Skip to content

Conversation

@vivi-enne
Copy link
Contributor

@vivi-enne vivi-enne commented Jul 14, 2025

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

  • Only needed for 7 vs 7. In 5 vs 5 both positions collide.

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_kickoff in the bevyhavior simulator and there should still not be a collision in golden_goal.

@github-project-automation github-project-automation bot moved this to Request for Review in Development Jul 14, 2025
@schluis schluis self-assigned this Jul 14, 2025
Copy link
Contributor

@schluis schluis left a 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 🤔

Comment on lines 101 to 108
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)
Copy link
Contributor

@schluis schluis Jul 15, 2025

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.

Copy link
Contributor Author

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.

@schluis schluis assigned vivi-enne and unassigned schluis Jul 19, 2025
@schluis schluis moved this from Request for Review to In Progress in Development Jul 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

post-game task: 4 on 7 collisions during walk-in

2 participants