Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Vision detection merge into main #5

Merged
merged 52 commits into from
Oct 7, 2024
Merged

Vision detection merge into main #5

merged 52 commits into from
Oct 7, 2024

Conversation

Anonymadly
Copy link
Member

Features :

  • Vision has been fixed (had some cases of null data causing code crashes)

  • Note Detection has been added

    • Detection is accurate up to 3 meters from the camera, decently trustworthy up to 4
    • Detection data is filtered over the previous 15 data points to mitigate shakiness
    • Stale data is removed after 1.5 seconds
  • Vision and Detection now run on separate threads from the main robot loop

  • Bot can face SPEAKER or a Note while driving

    • The PID object for this SwerveRequest wasn't accessible despite being a public variable so an extended class is used as a workaround
  • PathPlanner is tuned and so are other bot constants like max speed

  • Control scheme for swerve is reworked and explained in RobotContainer.java

I think that's it.

@Anonymadly Anonymadly self-assigned this Oct 7, 2024
@Anonymadly Anonymadly requested a review from Maxb0tbeep October 7, 2024 02:51
@Anonymadly Anonymadly added the enhancement New feature or request label Oct 7, 2024
@Maxb0tbeep Maxb0tbeep merged commit 82f0c0f into main Oct 7, 2024
2 checks passed
@Anonymadly Anonymadly deleted the vision-detection branch October 7, 2024 02:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants