-
Notifications
You must be signed in to change notification settings - Fork 98
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
Model submission for Gagarin sensor configuration 1 by CERBERUS #518
Model submission for Gagarin sensor configuration 1 by CERBERUS #518
Conversation
The initial assessment of the CERBERUS Gagarin model is complete. No further action is required at this time. You can track the progress of your submission in PR #521 once this one is merged. If you have any late additions, please submit a new PR to |
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.
looks good to me, only a few minor comments. Sensor data look fine in rviz, tf tree also looks good, and battery life measures ~5min sim time as per spec.
submitted_models/cerberus_gagarin_sensor_config_1/specifications.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Mihir Kulkarni <[email protected]>
Signed-off-by: Mihir Kulkarni <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
@mihirk284 , I've adjusted the starting z-offset of Gagarin in order to support marsupial pairing.See ntnu-arl#1. |
…sor_config_1 Fix z offset, and cleanup
Thanks for the submission @mihirk284 . I've moved these models to #527 and will close this PR. |
Model Submission for the CERBERUS Gagarin with Sensor Configuration 1.