Skip to content

decide for kicking depending on distance of seen robots#1007

Open
cleWu03 wants to merge 11 commits into
mainfrom
update/DribbleOrKick_with_robot_distance
Open

decide for kicking depending on distance of seen robots#1007
cleWu03 wants to merge 11 commits into
mainfrom
update/DribbleOrKick_with_robot_distance

Conversation

@cleWu03

@cleWu03 cleWu03 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

dribble if there are other robots close, since kicking tools to long

Proposed changes

Related issues

Checklist

  • Run pixi run build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

@github-project-automation github-project-automation Bot moved this to 🆕 New in Software Jul 4, 2026
@cleWu03 cleWu03 requested review from Flova and ayin21 July 4, 2026 13:36
@MegaIng MegaIng mentioned this pull request Jul 7, 2026
@jaagut jaagut marked this pull request as ready for review July 12, 2026 04:14
self.closest_robot_infront_dist = dist_to_robot
else:
if dist_to_robot < self.closest_robot_behind_dist:
self.closest_robot_behind_dist = dist_to_robot

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is infront and behind the correct term here? The other robot is not infront or behind relative to our own player (it's rotation is not relevant here).

@github-project-automation github-project-automation Bot moved this from 🆕 New to 👀 In review in Software Jul 12, 2026
@MegaIng MegaIng self-requested a review July 12, 2026 07:30
@jaagut

jaagut commented Jul 12, 2026

Copy link
Copy Markdown
Member

Tests don't pass for some reason

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

3 participants