Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove gate from DS prepare message #21

Merged
merged 2 commits into from
Feb 14, 2019

Conversation

morxa
Copy link
Member

@morxa morxa commented Jan 29, 2019

As discussed at today's TC meeting, we no longer want the gate to be part of the DS prepare message. We already have the order ID (since #1), therefore sending the gate is redundant.

Also remove some redundant information in the rulebook about DS preparation to avoid inconsistencies.

morxa added 2 commits January 29, 2019 16:01
The prepare message is explained later again, no need to state the
necessary information twice.
Copy link
Member

@timn timn left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@timn
Copy link
Member

timn commented Jan 31, 2019

+1

@thomerz
Copy link

thomerz commented Jan 31, 2019

+1

1 similar comment
@MostafaGomaa
Copy link
Member

+1

@teamsolidus
Copy link
Contributor

+1

1 similar comment
@morxa
Copy link
Member Author

morxa commented Feb 14, 2019

+1

@morxa
Copy link
Member Author

morxa commented Feb 14, 2019

Proposal accepted

  • in favor: 4
  • abstentions: 3
  • opposed: 0

@morxa morxa merged commit fa84e6d into master Feb 14, 2019
@morxa morxa deleted the thofmann/no-gate-in-ds-prepare-msg branch February 14, 2019 08:01
@morxa morxa added accepted-rule-change The rule change has been accepted and removed vote-in-progress labels Feb 14, 2019
@morxa morxa added this to the RoboCup 2019 milestone Feb 14, 2019
morxa referenced this pull request in robocup-logistics/rcll-refbox Apr 4, 2020
Mark the slot id and name as reserved so they won't be re-used on
accident.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted-rule-change The rule change has been accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants