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

Port to ROS 2 #99

Open
AndyZe opened this issue Jul 29, 2021 · 15 comments · May be fixed by #101
Open

Port to ROS 2 #99

AndyZe opened this issue Jul 29, 2021 · 15 comments · May be fixed by #101
Assignees

Comments

@AndyZe
Copy link
Member

AndyZe commented Jul 29, 2021

I'm starting to port this to ROS2. It is needed.

@JStech or @RoboticsYY, would you mind creation of a foxy branch?

@JStech
Copy link
Contributor

JStech commented Jul 29, 2021

Sounds good, thanks.

The dependencies need to be ported to ROS2: crigroup/baldor#5 crigroup/criutils#5 (crigroup/handeye will probably also need one).

@henningkayser henningkayser self-assigned this Nov 2, 2021
@henningkayser henningkayser changed the title Create ROS2 branch Port to ROS 2 Nov 2, 2021
@henningkayser
Copy link
Member

Abi took over this effort in #101, so I'm reassigning this to her.
There is also a ros2 branch now which should be used instead of foxy.

@Abishalini any thoughts on the WIP ports of the crigroup dependencies?

@henningkayser henningkayser linked a pull request Nov 2, 2021 that will close this issue
@lchojnacki-agribot
Copy link

Hello, any news about porting moveit_calibration package to ros2? I'd like to use this package in galactic, and for that reason I pulled PR version of it and compiled, but unfortunately I receive some 'symbol lookup error undefined symbol' errors.

If there is any way to move it forward I can help.

@Abishalini
Copy link

Hi @lchojnacki-agribot,
@vatanaksoytezer pushed a fix to rviz_visual_tools yesterday that fixes the symbol lookup error. If you run into more errors/issues, can you mention them here? Thanks!

@lchojnacki-agribot
Copy link

Hi @Abishalini,
Thanks for your response. The next issue is the following, after adding the plugin in rviz I get
image

I've installed everything via rosdep.

@tlpss
Copy link

tlpss commented Feb 2, 2022

hi @Abishalini, is there any news on the ROS2 port?

@tlpss
Copy link

tlpss commented Feb 4, 2022

Hi @Abishalini, Thanks for your response. The next issue is the following, after adding the plugin in rviz I get image

I've installed everything via rosdep.

@Abishalini, I'm encountering the same issue. The GUI loads on RVIZ but it cannot find plugins.
image

Would love to see see the calibration working in galactic and would be willing to assist when given some directions (I have no experience with RVIZ plugins)

@tlpss
Copy link

tlpss commented Feb 14, 2022

@henningkayser @Abishalini, ping for a small update w.r.t. ROS2

@HamzaNaseerFHWS
Copy link

@tlpss were you able to get it running on galactic? Im encountering the same issue at the moment.

@tlpss
Copy link

tlpss commented Apr 12, 2022

@HamzaNaseerFHWS , no I didn't unfortunately

@benediktkreis
Copy link

Is there any progress on the port to ROS2? I've noticed that in the foxy branch as well as in the ros2 branch, there are still catkin dependencies instead of colcon. As a result it is not possible to build the two branches with ROS2. Is this going to change in the near future? I believe that this is a crucial package for anyone working with CV and Moveit2.

@ozangungortuhh
Copy link

Hello, couldn't agree more with @benediktkreis. Is there any update on this? Me and my team would like to use this package in ROS2.

@peterdavidfagan
Copy link
Member

peterdavidfagan commented Apr 14, 2023

@ozangungortuhh did you perform this calibration on an earlier version of ROS while waiting for this to get ported? I think that is currently the strategy to take. I am also interested in using this package in ROS 2.

It may also be worth testing/contributing to #101.

@ozangungortuhh
Copy link

Hello @peterdavidfagan, I did the calibration with ROS 1 and used the result of the calibration launch file with ROS 2.

@hamsadatta
Copy link

Hello, any updates on this? I am interested in using this package in one of our projects.

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 a pull request may close this issue.