-
Notifications
You must be signed in to change notification settings - Fork 629
add python wrap for estimate_tag_pose #417
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
Conversation
|
Thanks. Can you show a side-by-side comparison of an C++ example and the equivalent Python example for detecting a tag and estimating its pose with the new wrapper? |
I add test_tag_pose_erstimation case For c interface For py interface |
|
For case 33369213973_9d9bb4cc96_c py interface log |
|
Most of your files are missing a new line at the end (see the red symbol in the change overview). Can you add these? |
Sure I never used clang as compiler, this is a new knowledge for me |
|
I checked "Allowing edits by maintainers" If I make format or other minor problem again, please help me update |
This is just the style check and independent of the compiler. |
8fdec47 to
7dc8384
Compare
I managed to squash your EOL fixup commits and push to your PR branch. |
|
Thanks. |
#415
Told you I'm going to try this
Please review