Skip to content

Kick foot decision biased goal position while going to ball #675

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

cleWu03
Copy link
Contributor

@cleWu03 cleWu03 commented Mar 14, 2025

Summary

Calculate the offset for the go to ball position depending on the relative position to the ball

Proposed changes

Related issues

Checklist

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

@@ -22,9 +24,13 @@ def __init__(self, blackboard, dsd, parameters):
self.blocking = parameters.get("blocking", True)
self.distance = parameters.get("distance", self.blackboard.config["ball_approach_dist"])
# Offset so we kick the ball with one foot instead of the center between the feet
self.side_offset = parameters.get("side_offset", 0.08)
self.side_offset = parameters.get("side_offset", 0.00)

Copy link
Member

Choose a reason for hiding this comment

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

Move foot selection here

@jaagut jaagut moved this from 🆕 New to 👀 In review in Software Mar 15, 2025
@jaagut jaagut requested a review from Flova May 4, 2025 12:54
@jaagut
Copy link
Member

jaagut commented May 4, 2025

@cleWu03 I've just re-requested a review from @Flova, so he gets notified again.

@Flova
Copy link
Member

Flova commented May 4, 2025

We did some testing during the Go, which still had some issues iirc.. Maybe @cleWu03 remembers more or we can test it in sim.

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.

4 participants