Skip to content
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

spec: Should fix ambiguities for //inertial/{pose,mass,inertia} #70

Closed
2 tasks done
EricCousineau-TRI opened this issue Apr 17, 2022 · 6 comments
Closed
2 tasks done
Assignees
Labels
bug Something isn't working

Comments

@EricCousineau-TRI
Copy link
Collaborator

EricCousineau-TRI commented Apr 17, 2022

Ambiguities raised by @mitiguy in email thread. Relates #18

Items:

@EricCousineau-TRI EricCousineau-TRI added the bug Something isn't working label Apr 17, 2022
@EricCousineau-TRI
Copy link
Collaborator Author

Made changes to URDF as seen here: https://wiki.ros.org/action/diff/urdf/XML/link?action=diff&rev1=76&rev2=77

@scpeters
Copy link
Member

Made changes to URDF as seen here: https://wiki.ros.org/action/diff/urdf/XML/link?action=diff&rev1=76&rev2=77

I made a minor to fix to one of your added sentences: https://wiki.ros.org/action/info/urdf/XML/link?action=diff&rev2=78&rev1=77

otherwise it looks good

@scpeters
Copy link
Member

draft pull request to fix documentation source for sdformat.org

@scpeters
Copy link
Member

scpeters commented May 6, 2022

I have merged gazebosim/sdformat#990 and uploaded the revised specification files

http://sdformat.org/spec?ver=1.9&elem=link#link_inertial

I believe the only remaining action item is ensuring that it renders properly on the website

@EricCousineau-TRI
Copy link
Collaborator Author

Deferring website rendering to internal web app issue 51

@mitiguy
Copy link

mitiguy commented May 19, 2022

Thanks Eric and all. Nice work. Much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants