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

Add ModelCalibrationHelper class #576

Merged
merged 1 commit into from
Sep 13, 2019
Merged

Conversation

traversaro
Copy link
Member

The iDynTree::ModelCalibrationHelper class simplifies the process of loading a model from file, update its inertial parameters and exporting again to file. It is useful in the process of calibration of the inertial parameters of a model.

@traversaro
Copy link
Member Author

As part of the PR, both the release notes and test for the new class have been added.

@traversaro
Copy link
Member Author

@nunoguedelha @fjandrad you may find this class useful for future works.

@traversaro
Copy link
Member Author

As this function can also be used with human models (with inertial parameters obtained from tables or calibration) this can be of interest also of AnDy people @Yeshasvitvs @lrapetti .

Copy link
Member

@diegoferigo diegoferigo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this function can also be used with human models (with inertial parameters obtained from tables or calibration) this can be of interest also of AnDy people @Yeshasvitvs @lrapetti .

Indeed! Particularly, the serialization to string can be useful if we want to implement an online model generator containing also an RPC server that sends the runtime-computed model to the devices part of the architecture. @Yeshasvitvs @lrapetti

@traversaro
Copy link
Member Author

Indeed! Particularly, the serialization to string can be useful if we want to implement an online model generator containing also an RPC server that sends the runtime-computed model to the devices part of the architecture. @Yeshasvitvs @lrapetti

At the moment the only method to "update" the model is meant to specify the inertial parameters, but we can also add other methods to change kinematic information (for example for kinematic calibration). It could also be helpful for sensor calibration, even if sensors export in URDF still needs to be implemented, see #536 and #554 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants