pal-robotics / aruco_ros

Software package and ROS wrappers of the Aruco Augmented Reality marker detector library
MIT License
449 stars 307 forks source link

Node for publishing all visible markers #12

Closed jdlangs closed 9 years ago

jdlangs commented 9 years ago

Is the plan to make this the default node that people are directed to and then add the filtering functionality? Maybe there can be an Indigo/Jade official release with this behavior?

bmagyar commented 9 years ago

I'll test it tonight and if all are well, I'll merge it right away. The answers to your questions are in order are as follows: yes, yes :) I am planning to make this the main executable, maybe even to eliminate the old ones.

bmagyar commented 9 years ago

Apart from the launch file typo I mentioned above, all seems fine. Once this is merged I will continue working on it during next week in order to reach a point where it is releasable.

Thanks for the contribution!

jdlangs commented 9 years ago

Yep, sorry about that. I did some final renaming but forgot about the launch file. It should be all set now.

bmagyar commented 9 years ago

Merging... :+1:

nbaddorf commented 3 years ago

Hey, thanks so much for you work on this! I have spent the past couple of days trying to get this system to work, and I finally achieved it, but I have, probably a simple issue, where the /aruco_marker_publisher/markers topic is outputting: `header: seq: 695 stamp: secs: 1614385501 nsecs: 30457688 frame_id: "eye" markers:

nbaddorf commented 3 years ago

Sorry, about the bold text in the previous msg, the dash that is causing my problems, is making the github code tag not work, and I cant seem to fix it. But for clarification there is a "-" in the output between the "markers:" and the "header". Thanks!

v-lopez commented 3 years ago

How are you subscribing to the topic? C++ or Python subscriber?

nbaddorf commented 3 years ago

Hi, I was trying to subscribe with python. In the end I was able to figure out that if I made a new variable with the condense of msg.markers[0], it would let me use that new var to read: newMsg.id and newMsg.pose. Thanks for making this system!

Sent from my iPhone

On Mar 1, 2021, at 06:20, Victor Lopez notifications@github.com wrote:

How are you subscribing to the topic? C++ or Python subscriber?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.