Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

update description of challenge field to refbox changes #101

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

TarikViehmann
Copy link
Contributor

This change is implemented through robocup-logistics/rcll-refbox#196.

Originally, the challange track fields were played on magenta-halve, because the refbox generated fields on magenta and mirrored them to cyan.
Machines for both teams were placed on top of eachother, so team cyan could play on the magenta halve.

There were 2 issues with this:

  1. The field generation constraints were actually for team cyan and interpreted as team magenta machines causing errors regarding legal rotations for delivery stations.
  2. Some participants prefer to not receive machine info for machines that are placed on top of eachother.

To address this, actually treat the generated field correctly as cyan field, mirror it to magenta regardless and therefore let teams divide whether to play as cyan in the cyan half or as magenta in the magenta half.

@TarikViehmann TarikViehmann added rule-change A proposed change to the rules vote-in-progress labels Feb 24, 2024
@TarikViehmann TarikViehmann added this to the RoboCup 2024 milestone Feb 24, 2024
@snoato
Copy link
Contributor

snoato commented Mar 6, 2024

+1

@TarikViehmann TarikViehmann requested a review from snoato April 5, 2024 15:26
Copy link
Contributor

@snoato snoato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reasonable change but figure should be updated

@TarikViehmann TarikViehmann merged commit 6a5fe0f into master Apr 10, 2024
3 checks passed
@TarikViehmann TarikViehmann deleted the tviehmann/challenge-field-update branch April 10, 2024 11:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
rule-change A proposed change to the rules vote-in-progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants