-
Notifications
You must be signed in to change notification settings - Fork 46
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
feat: support Aeva Aeries II #169
base: develop
Are you sure you want to change the base?
Conversation
a8c9ee5
to
4829c22
Compare
4829c22
to
282c6aa
Compare
🟢 Evaluation🟢 Pointcloud Output
*: red: strongest, magenta: second strongest. 🟢 PerformanceThe pointcloud is published consistently at around 10 Hz: 🟢 Hardware Monitor
***: 🟢 PTP & SynchronizationPTP has been tested with all provided mechanisms (1588v2, gPTP, automotive) and works with all of them using the vendor-provided configurations. Pointcloud timestamps take around 1-2 min to converge.
🟢 Error Handling & RecoveryCurrently, Nebula does not attempt to recover from communication errors and instead exits, instead of continuing in a possibly corrupted state.
All of these errors only occur during startup. No crashes / errors were observed during runtime. 🟢 ParametersAll parameters from the sensor's manifest that have more than one possible value have been added to Nebula. No attempt was made to convert them to the common naming scheme used in the other drivers (e.g. Validation:
|
@drwnz All issues from yesterday have been addressed and I could not find any bugs or unexpected behavior since. I think this is ready to merge. |
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #169 +/- ##
==========================================
Coverage ? 14.61%
==========================================
Files ? 103
Lines ? 9207
Branches ? 1540
==========================================
Hits ? 1346
Misses ? 6767
Partials ? 1094
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -0,0 +1,10 @@ | |||
<?xml version="1.0"?> | |||
<launch> | |||
<arg name="sensor_model" description="Aeries2"/> |
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.
description or default?
…, higher timeouts
Co-authored-by: David Wong <[email protected]>
b3ffe46
to
e2da3f4
Compare
PR Type
Description
Add support for Aeva Aeries II:
/diagnostics
Review Procedure
rviz2
)launch_hw:=true
, subscribed to whenlaunch_hw:=false
rqt
's Plugins->Robot Tools->Runtime Monitor)Remarks
This is a fairly minimal implementation for now.
Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.
Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.
Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.
CI Checks