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

Move all changes from fresno comp to main so we can continue working off of main instead of fresno comp (also since this is the updated code) #110

Merged
merged 22 commits into from
Mar 28, 2024

Conversation

maxspier
Copy link
Member

No description provided.

qntmcube and others added 20 commits March 22, 2024 15:24
Co-authored-by: Lydia Honerkamp <[email protected]>
… until we'd expect it to be set) (#103)

* harden camera/visionspeakerhelper code.  delay when we get the alliance and protect against other potential initialization errors

* update more alliance logic

* set path to what path.flipPath() returns

* address PR comments

* Spotless fixes

* Skip auton if can't determine flipping

* linting

---------

Co-authored-by: Ryan Cahoon <[email protected]>
…eting code in OI (just no way to trigger it now)

* code for reverting to old shooting code

* lint all code

* integrated with driver OI

* apply linter to pass CL

* add a stop drive

* 54 point match!
@maxspier maxspier requested review from 1yd1a, qntmcube and rcahoon March 26, 2024 18:15
Copy link
Contributor

@1yd1a 1yd1a left a comment

Choose a reason for hiding this comment

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

I think we might need to get a better understanding of our bandwidth issues before we completely delete logs from main. Lmk what you think

Copy link
Contributor

@qntmcube qntmcube left a comment

Choose a reason for hiding this comment

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

LG, should add the boxop complete control of shooter changes and revamped logs soon

@maxspier
Copy link
Member Author

I think we might need to get a better understanding of our bandwidth issues before we completely delete logs from main. Lmk what you think

Personally, since the logs being commented out won't hurt anything on main, IMO it should be fine to merge as is. As Raj points out as well we can simply do this later - it doesn't hurt anything.

@maxspier maxspier merged commit 2a1d8f6 into main Mar 28, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants