-
Notifications
You must be signed in to change notification settings - Fork 2
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
Merge rewrite into master #42
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Create Selector class * Switch back to procedural programming * Add label to display current auton. Change to dark theme * Add radio button functionality * Simplify text update, remove autonMap * single-screen selector --------- Co-authored-by: Andrew Lu <[email protected]>
--------- Co-authored-by: Andrew Lu <[email protected]>
* Using mutex instead of bool semaphore * Cleaner locking * Changed all variables to atomic * Include rtos for mutex * Test cases * Use mutex pointer * Remove atomic Builds, will readd atomic later * Print to brain * Mostly fixed atomic prev_pose doesn't like being atomic * Use mutex for prev_pose * Use unique lock * Testing * Removed atomic (fixed data abort) & mutex works for abstract localizer * Fix atomic (again) * Removed debug prints * Clean up main.cpp --------- Co-authored-by: Mihir Laud <[email protected]>
* Add validation code for ADILocalizerBuilder * Fix build validation code. Add testing code * Add safeguards to ADILocalizer update function * Removed testing code
* settle_time * Fix formatting * Add default for settle_time. Fix angular settling * Fix IDE warning --------- Co-authored-by: Mihir Laud <[email protected]>
* work * Add settle time to Boomerang --------- Co-authored-by: Mihir Laud <[email protected]> Co-authored-by: Mihir Laud <[email protected]>
* Add IME Support for Odom * Added some protections on the builder - CANNOT TEST CUZ LIBLVGL Broken * Add testing code. Resolve IDE warning * Added ability to use IMU on ADI and IME Localizers Check Voss working thread for more details about what is working/broken * Remove testing code. Add safeguards for IMELocalizer update * Fix naming * Fix boomerang naming --------- Co-authored-by: cjsport11 <[email protected]>
* Create override functions for set_pose in localizers * Add IMU checks * change set_heading to set_rotation --------- Co-authored-by: Andrew Lu <[email protected]>
* Added get motor function * Mark the getMotor function const and remove it from cpp file * Add getMotor example to main.cpp
* commit * add settling when stopped, fix relative movements * Refactor flags --------- Co-authored-by: Andrew Lu <[email protected]>
* commit * add settling when stopped, fix relative movements --------- Co-authored-by: Mihir Laud <[email protected]>
Co-authored-by: Mihir Laud <[email protected]>
* refactor includes, add clang-format * create github action * Committing clang-format changes --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
mihirlaud
approved these changes
Jan 29, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It looks like we're at a stable point now, so I think we can merge the rewrite branch into master.