-
Notifications
You must be signed in to change notification settings - Fork 8
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
Jazzy 2.0 RC #122
Merged
Merged
Jazzy 2.0 RC #122
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
Update CI to use Jazzy instead of Humble
Sync initial development changes for Jazzy
* Generate can bridges * Generate script source robot workspace * Remove extra line --------- Co-authored-by: Roni Kreinin <[email protected]>
Fix tests for Jazzy
* Add phidget spatial config and launch files * Add dependency * Double to single quotes
* Add OAKD * Remove unused parameter * Add UDEV for OAKD
#84) * [clearpath_robot] Added script to grab diagnostic logs for troubleshooting. * Make grab-diagnostics script executable and installed --------- Co-authored-by: Luis Camero <[email protected]>
…rate is always overridden, must be integer
* Added manual launch files for encoding/decoding ffmpeg
* Added lynx hardware interface * Lynx motor driver Rename clearpath_platform namespace to clearpath_hardware_interfaces * Added A300 and Inventus battery to generator * A300 lighting * Dependencies and README * Rename platform to hardware_interfaces in hardware.xml * Fix append of bms in generator * Removed wheel_joints_ map --------- Co-authored-by: Luis Camero <[email protected]>
* Inventus CAN module ids depending on battery configuration * Name each socketcan interface launch file by interface name * Pass filename argument to launchfile
…enerator into the robot generators (#110)
* Send boot request to all drivers on action start * Removed first ack check * Added Ack counter * Resend data if counters dont match * Make app counter a private variable of driver * Re-attempt entering bootloader and alive check 5 times before skipping * Fix for skipping unresponsive Lynx's * Removed whitespace
* Set NeedsReset to higher priority than Stopped Changed NeedsReset lighting pattern Set battery message initial values to not trigger battery fault Increased low battery lighting sequence duration * Updated low battery sequence
…ludes to reference this file (#115)
* Add zenoh service files & generators
…clearpath_offboard_sensors (#118)
luis-camero
requested review from
civerachb-cpr and
roni-kreinin
and removed request for
a team
January 14, 2025 20:43
I'd like to get #120 merged and add the latest lynx firmware before we merge this. |
civerachb-cpr
approved these changes
Jan 16, 2025
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.
I'm responsible for most of these changes, so we may want another reviewer. But these are the changes I've tested on cpr-dozer.
* Update motor error lighting sequence if motor_states changes without the lighting state changing Added lighting emulator script * Removed light emulator
… check from a workspace in generate. (#127)
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.
No description provided.