-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add minimal example, remove creopyson test #76
Conversation
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.
Since now 2bars can be used we could add a "Test your installation" section where the user have to try to export the urdf of 2bars
src/creo2urdf/src/Utils.cpp
Outdated
* - The first element is a boolean indicating success (true) or failure (false). | ||
* - The second element is an iDynTree::Direction representing the unit vector of the specified axis | ||
* transformed into the coordinate system of the specified link frame. If the operation fails, | ||
* this vector will be zero-initialized. | ||
*/ |
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 would move the doxygen documentation to the Utils.h, if I recall correctly the doxyfile has a filter for getting only the .h/.hpp
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.
removed it from the cpp, for some reason it was there.
} | ||
} | ||
} | ||
|
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.
This is not used anywhere since we refactored the code for using the ElementTree right?
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.
Exactly!
This PR adds a Hello World example of creo2urdf by using the good old 2bars assembly. The yaml contains only the mandatory stuff.
Additionally: