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

Reset detection time in referee_pose_detection_filter #1333

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

Conversation

JackyBloxx
Copy link
Contributor

Why? What?

fixes that we skip standbyphase in 2. half if we didnt redeploy in halftime so we dont get penaltys for motion in standby

Fixes #1278

ToDo / Known Issues

Ideas for Next Iterations (Not This PR)

How to Test

see if it happens

@JackyBloxx JackyBloxx self-assigned this Jul 18, 2024
@JackyBloxx JackyBloxx removed their assignment Jul 18, 2024
@julianschuler julianschuler self-assigned this Jul 19, 2024
@JackyBloxx JackyBloxx force-pushed the sacrifivial_lamb_issue branch from c9e2105 to 4a1d0f7 Compare August 7, 2024 14:45
@MaikRe
Copy link
Contributor

MaikRe commented Oct 18, 2024

Whats the state of this? Is a 'merge when ready' all that is missing? @JackyBloxx

@JackyBloxx
Copy link
Contributor Author

Whats the state of this? Is a 'merge when ready' all that is missing? @JackyBloxx

the problem is it isnt tested so i doesnt realy now if it realy fixed the probelm.

@pejotejo
Copy link
Contributor

@oleflb du willst dinge testen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Request for Review
Development

Successfully merging this pull request may close these issues.

Sacrificial lamb issue in second half
5 participants