-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Clearpath Heron USV #6548
Clearpath Heron USV #6548
Conversation
There are several errors reported by the CI on the "Test Sources". In particular:
Can you please fix them? |
I fix that bugs @omichel |
Thank you. Could you please also write a short documentation? This is required for robots distributed in Webots. |
@omichel I finish all of for document, can you check ? |
okey sir. |
mr. @omichel , can you check ? |
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.
As mentioned, I would also appreciate if you can fix the icon which should be a screenshot of the simulated robot, not the real one.
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.
Please see my latest comment.
mr. @omichel, can you review ? is it all of done ? |
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.
As already requested, please remove the linearVelocity
and angularVelocity
useless fields from all the world and proto files.
Sir, removed linearVelocity and angularVelocity. |
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.
Thank you.
Sir, Thank you very much indeed for your feedback. |
Description
Clearpath's USV vehicle named Heron has been added.
Tasks
Add the list of tasks of this PR.
Screenshots