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

Aruco 3.0.4 #44

Open
wants to merge 25 commits into
base: kinetic-devel
Choose a base branch
from
Open

Aruco 3.0.4 #44

wants to merge 25 commits into from

Conversation

v-lopez
Copy link
Contributor

@v-lopez v-lopez commented Feb 7, 2018

No description provided.

@v-lopez v-lopez requested a review from bmagyar February 7, 2018 11:35
@v-lopez v-lopez force-pushed the aruco-3.0.4 branch 4 times, most recently from 09d8177 to 4a2cc03 Compare February 7, 2018 12:26
@v-lopez
Copy link
Contributor Author

v-lopez commented Feb 12, 2018

@bmagyar Any feelings about this?

@bmagyar
Copy link
Collaborator

bmagyar commented Feb 13, 2018

A code change of this magnitude cannot be reviewed in the usual way.
If performance and detection rates are still ok then we should be generally fine.

There's one thing I've spotted though: the optimal markers tool seems to be gone. What's the recommended way to generate new (sets of) markers now?

@v-lopez
Copy link
Contributor Author

v-lopez commented Feb 20, 2018

@bmagyar I haven't been able to find a substitute for the selectoptimal markers tool in the latest aruco version.

@v-lopez
Copy link
Contributor Author

v-lopez commented Jan 21, 2020

@bmagyar I'm working on some more major changes on top of this. At this point I'm torn between setting this branch as melodic-devel, putting all in a fork and leave this intact, or making an aruco3_ros package or something, starting from this.

I want to make quite a bit of backward incompatible changes in the launch files that are due from long time ago, like allowing full topic names as input of launch files, instead of "side", removing hard coded frames.. Also restructuring the code and making some functions reusable from outside the nodes in a library.

What are your thoughts?

@bmagyar
Copy link
Collaborator

bmagyar commented Jan 21, 2020 via email

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.

4 participants