Closed sarthou closed 6 years ago
Could you rebase to the origin, which has been slightly updated? CI failed and I see memory allocation error, which may or may not be already addressed by the latest change.
It sounds better but I do not understand why some CI have failed
Normally, I just added some small correction and I removed a lot of warning in the ABI so these changes are interesting in my opinion, so I think it would be better to re-target this PR on the branch Melodic
No problem for the README file, I'll do it
Really sorry for spaces, my text editor does it automatically ... I'll be more careful next time
Okay, then I just created melodic-devel
branch and made it a default for this repo. I have to ask you to re-open a PR targeting there, as I don't think there's a feature to change the target branch on this PR. Thanks @sarthou!
Note melodic-devel
is based off from kinetic-devel
, so you should be able to use the same branch of yours and just change the target branch when you open a new PR.
This pull request does not affect the classic behavior of ar_track_alvar but adds two utilities that allow you to simply create remote markers (on XYZ) and cubes with six markers.
Based on sampleMarkerCreator, it generates the PNG and XML files with the corresponding transformations.
In addition, it allows users to create color markers for more diversity.