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

More dense candidates #1399

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

More dense candidates #1399

wants to merge 10 commits into from

Conversation

h3ndrk
Copy link
Member

@h3ndrk h3ndrk commented Jul 20, 2024

Why? What?

  • More dense candidate circles s.t. we have better fitting candidates for the neural networks
  • There must be at least 3 segments on a circle to become a candidate
  • Fix field border drawing to be above the field border seeds

ToDo / Known Issues

None

Ideas for Next Iterations (Not This PR)

  • Fix and reenable test cases of perspective grid candidate generator

How to Test

Use the ball detection image overlay or/and ball candidates panel in Twix to see the changes. We should detect the ball better or similar as before.

@h3ndrk h3ndrk enabled auto-merge July 20, 2024 20:10
@h3ndrk h3ndrk disabled auto-merge July 20, 2024 20:17
@h3ndrk h3ndrk force-pushed the more-dense-candidates branch 2 times, most recently from 9fc0887 to 0fa8dca Compare July 21, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Request for Review
Development

Successfully merging this pull request may close these issues.

None yet

1 participant