-
Notifications
You must be signed in to change notification settings - Fork 57
avoid collision between striker and striker supporter #1934
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems as this could work 🤔
| let is_opponent_kickoff = if let Some(filtered_game_controller_state) = | ||
| &world_state.filtered_game_controller_state | ||
| { | ||
| filtered_game_controller_state.kicking_team == Some(Team::Opponent) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to only apply during a kickoff. See here: determine_playing_situation in kick_selector.rs.
This also triggers, if a kickoff during the game occurs, and not exclusively, if the robots are walking into the field from the sidelines. I think you have to find a way to distinguish this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks. I hope it is fixed now. I now check also for the first kick_off, which is after initial.
3feae70 to
24b022d
Compare
…he field Co-authored-by: phillip kammradt <phillip.kammradt@tuhh>
24b022d to
1ded25e
Compare
Why? What?
To avoid collisions between striker and striker supporter when entering the field for opponent kickoff, the path of the striker supporter (4) is split into two segments depending on a path checkpoint.
Fixes #1298
ToDo / Known Issues
Ideas for Next Iterations (Not This PR)
If there are some improvements that could be done in a next iteration, describe them here.
How to Test
We tested it in the bevyhavior simulator so far because we need a field. Run the
golden_goal_opponent_kickoffin the bevyhavior simulator and there should still not be a collision ingolden_goal.