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

Initial DH Chain Kinematic Calibration functionality #60

Merged
merged 12 commits into from
Jul 21, 2020

Conversation

marip8
Copy link
Collaborator

@marip8 marip8 commented Jun 17, 2020

Description

This PR implements a number of changes to support a minimum working example of calibration of a kinematic chain based on DH parameters with camera and target

  1. Updates the DH chain class to work well with Ceres auto diff cost functions
    • Template specific methods (getFK and getRelativeTransform) to work easily with auto diff
    • Use Eigen vectors to hold data that needs to be cast between doubles and jets
    • Add the ability to specify DH parameter offsets that will be calibrated by Ceres
    • Added function for getting the DH parameters from a chain in an N x 4 matrix
    • Removed GaussianNoiseDHTransform since virtual class functions cannot be templates
  2. Defined a kinematic calibration observation structure
  3. Created functions for generating kinematic calibration observations
    • Added a function for creating a DH chain with randomly perturbed parameters based on an input chain to replace the GaussianNoiseDHTransform class
  4. Created initial kinematic calibration algorithm
  5. Created unit tests for kinematic calibration algorithm
    • Single kinematic chain with camera transform, target transform, and camera chain to target chain transform

Currently the kinematic calibration converges for the perfect cases represented in the unit test, as well as some perturbed cases that I tried. More development is necessary to finish this algorithm, but that should come in a later PR.

This PR replaces #53

TODO

@marip8 marip8 mentioned this pull request Jun 17, 2020
@marip8 marip8 force-pushed the feature/dh_chain_kin_cal branch 2 times, most recently from 6238d38 to 1485355 Compare June 30, 2020 15:19
@marip8 marip8 force-pushed the feature/dh_chain_kin_cal branch 2 times, most recently from 5d58913 to d5bc832 Compare July 15, 2020 14:43
@marip8
Copy link
Collaborator Author

marip8 commented Jul 21, 2020

@schornakj is this good to go now?

@schornakj
Copy link
Contributor

@marip8 Looks good to me!

@marip8 marip8 merged commit 69ed22d into Jmeyer1292:master Jul 21, 2020
@marip8 marip8 deleted the feature/dh_chain_kin_cal branch July 21, 2020 16:03
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.

2 participants