uuv_simulator icon indicating copy to clipboard operation
uuv_simulator copied to clipboard

Noetic release

Open Timple opened this issue 4 years ago • 36 comments

Would you mind releasing these packages also for the newest distribution ROS Noetic? :)

Timple avatar May 25 '20 06:05 Timple

I'm on it ;)

musamarcusso avatar May 25 '20 06:05 musamarcusso

Well that is very fast! :D

Timple avatar May 25 '20 07:05 Timple

Not all dependencies are released for noetic yet, e.g. joy and joy_teleop, needed by uuv_teleop. https://github.com/ros-drivers/joystick_drivers/issues/170 https://github.com/ros-teleop/teleop_tools/issues/51

arturmiller avatar Jun 03 '20 19:06 arturmiller

that is true, the first step would be to add a job on Travis so that we can check when the dependencies can be all found.

musamarcusso avatar Jun 05 '20 07:06 musamarcusso

I have done some experiments to update the Travis build. https://github.com/arturmiller/uuv_simulator/tree/noetic https://travis-ci.org/github/arturmiller/uuv_simulator/jobs/693993787 It is not easy to implement that, because industrial_ci has made big changes. There are no BEFORE_SCRIPT and AFTER_SCRIPT arguments anymore and they have moved to colcon instead of catkin. As I know catkin_tools is also not available for noetic, yet.

arturmiller avatar Jun 05 '20 07:06 arturmiller

yes, I saw that they made a legacy version available with the old setup. I did some experiments locally on my machine here, there are a few things to check yet.

musamarcusso avatar Jun 05 '20 07:06 musamarcusso

I am now able to build and test uuv_simulator with the recent industrial_ci changes: https://github.com/arturmiller/uuv_simulator/tree/noetic https://travis-ci.org/github/arturmiller/uuv_simulator/builds/696565670 Most changes are straightforward but, there is one issue, which requires some form of refactoring. The tests of uuv_gazebo_ros_plugins require uuv_descriptions. This results in a circular dependency. For now I have removed the tests in uuv_gazebo_ros_plugins. Apart from that I am not sure why the run_tests.sh script is used instead of the build in procedure from industrial_ci. I have used the build in test runner, because catkin_tools is not yet released in Ubuntu 20.04. It seems to work well. If you want to merge this, I would clean up my commits and create multiple issues and PRs, where we can discuss everything if necessary.

arturmiller avatar Jun 09 '20 19:06 arturmiller

I forgot to mention. The packages joy and joy_teleop are still not available. For now, to make the travis builds succeed, I have commented these dependencies in the package.xml. Building and testing don't require them. But it is still blocking the noetic release.

arturmiller avatar Jun 09 '20 19:06 arturmiller

@arturmiller Any updates? Can I use this repo for ROS noetic now since joy and joy_teleop are now available for ROS noetic ?

purvikpatel avatar Jul 20 '20 06:07 purvikpatel

Bump. Would be very interested to know if UUV Sim is good to go (or would be good with some work on the user side when installing) on Noetic. My place is fully on Python3 and I'm currently dealing with the headache of coercing Melodic to work with Python3.

ethanjpark avatar Jul 21 '20 13:07 ethanjpark

Yes joy and joy_teleop are available now for noetic. catkin_tools is still not properly ported, but it can be easily installed from source with pip. @purvikpatel I would not recommend to use my fork directly. The build is passing, but I have not done any manual testing. I am sure it wont work, as the test coverage is not very high. In addition this branch is full of "exploratory" commits, because i was not able to exactly duplicate the travis build locally. I have hoped for more interest from the maintainers on this, but they seem to be quite busy. My last issue #406 got nearly no attention. I would like to see this repo being updated, but I could also create a working fork for noetic and Python3 (if the interest in general is large enough and if the maintenance of the repo keeps missing). However, I would not be able to maintain this project further and I cannot release any debian packages.

arturmiller avatar Jul 23 '20 16:07 arturmiller

catkin_tools is still not properly ported, but it can be easily installed from source with pip.

Actually I think this is fixed. From ubuntu 20.04 I can simply do sudo apt install python3-catkin-tools

Timple avatar Jul 24 '20 06:07 Timple

Any update on the progress of this? I tried the repo @arturmiller made and it is giving me a free(): double free detected in tcache 2 error in gazebo.

blaine141 avatar Sep 01 '20 05:09 blaine141

catkin tools is operational in Noetic now. Is someone willing to release this for Noetic? :slightly_smiling_face:

Timple avatar Nov 09 '20 11:11 Timple

you are doing an amazing job! I thanks this actual release: https://github.com/uuvsimulator/uuv_simulator/pull/423/commits/66474bd3648f9fa2d0cf1d42e153ee08f3d3da7e#diff-71b515d752a471114f16806cd6bf541d62e8d81e79f4fbce6325c45613561015 for adapting the melodic version of uuv to noetic, can't wait for the official release. You are awesome. (posting this here for those trying desperately to make it work on noetic for the time being).

my-name-is-D avatar Nov 30 '20 08:11 my-name-is-D

Hey everyone. Until there is an official release for noetic, does anyone have a working fork of the repo or a set of instructions that can be used to install it? Appreciate all the awesome work being done here!

amaarquadri avatar Jan 24 '21 07:01 amaarquadri

check the link of the comment above ;) It's was added there to make your life easier.

my-name-is-D avatar Jan 24 '21 09:01 my-name-is-D

Thanks! I tried setting everything up based on that link and looking around the different pull requests and their comments. I got most of the way there but I'm getting the following error after building everything and trying to run it:

ERROR: cannot launch node of type [message_to_tf/message_to_tf]: message_to_tf

Also (not sure if its related), but Gazebo is just showing a black screen, even though my code is actually running.

I compiled the process that I used here: https://github.com/Waterloo-Aquadrone/aquadrone2020_dev_workspace/blob/master/noetic_setup.md.

There's a good chance that I might just be making a dumb mistake. I would really appreciate any help at all. Thanks so much for all the awesome work!

amaarquadri avatar Jan 26 '21 02:01 amaarquadri

well, as far as i know, message_to_tf refers only to a launch and a c file in uuv_description ~to double check~. If everything compiled you may assume you correctly installed uuv_simulation.

I invite you to ask this question in ros answers (check:https://answers.ros.org/question/67488/roslaunch-error-error-cannot-launch-node-of-type/) or stackoverflow.

my-name-is-D avatar Jan 26 '21 17:01 my-name-is-D

I am trying to implement this Neotic release and am running into a segfault when I add my robot. I have narrowed it down to whenever I add a sensor. I can spawn my robot without any sensors and it floats and works. The same code works in the melodic release but not in noetic.

Here's an example of a block of code that will cause a segfault. I'm almost certain namespace and inertial_reference_frame exist as other code works.

<xacro:include filename="$(find uuv_sensor_ros_plugins)/urdf/sensor_snippets.xacro"/>

<xacro:default_imu_macro
    namespace="${namespace}"
    parent_link="${namespace}/base_link"
    inertial_reference_frame="${inertial_reference_frame}">
    <origin xyz="0 0 0" rpy="0 0 0"/>
</xacro:default_imu_macro>

Gazebo just prints "Segmentation fault (core dumped)" and then the process dies. Does anyone have any ideas?

blaine141 avatar Feb 20 '21 05:02 blaine141

If anyone else is having this problem, add return true; to line 144 of ROSBasePlugin.cc. I also had lots of errors related to ned. As a temporary solution, I deleted lines 207 and 208 from UnderwaterObjectROSPlugin.cc

blaine141 avatar Feb 22 '21 04:02 blaine141

Bumping this thread again :) @musamarcusso will you or any other maintainer release these packages for Noetic in the near term? If not, or if you're busy, let me know how may I help.

hidmic avatar Jun 07 '21 13:06 hidmic

@hidmic, unfortunately, I don't have full access to the repository to manage the porting to Noetic, maybe waiting for some answer from @musamarcusso or working in some fork :/

pxalcantara avatar Jun 07 '21 13:06 pxalcantara

Hi @pxalcantara! Long time no see :)

maybe waiting for some answer from @musamarcusso or working in some fork

I see. Not forking (i.e. not splitting efforts across copies of this repo) would be best IMHO.

hidmic avatar Jun 07 '21 21:06 hidmic

Hello, I was curious if there were any changes/updates regarding the official noetic release? I have been using the diff @my-name-is-D linked above and have been tuning as time progressed, but am still having some issues with sensor integration - would be glad to contribute to the official migration as to learn more about the development of the platform as well!

onurbagoren avatar Oct 15 '21 15:10 onurbagoren

Unable to locate package ros-neotic-uuv-simulator I am getting this error while I am trying to use the uuv_simulator which is available for melodic but I have ubuntu 20.04 and neotic I am getting this error. It would be really great if this error is fixed as soon as possible .

Anishamohapatra6 avatar Nov 10 '21 15:11 Anishamohapatra6

Any update on a noetic release?

jake3991 avatar Mar 18 '22 16:03 jake3991

I am also interested in any updates of UUV to Noetic / Foxy

romulogcerqueira avatar Apr 14 '22 14:04 romulogcerqueira

Hi @my-name-is-D (sorry for bothering you), thanks for your work on the the Noetic release. I am facing problems when suing the "forward_looking_sonar" with Noetic. I have worked with the simulator using Kinetic ans everything was ok. In Noetic, the sensors is not reading properly. The image in RViz (see below) shows a wall, which is not in the environment. the sensor publish the data using the robot_name/points topic and the message is pointcloud2 type. Thanks in advance. Screenshot from 2022-05-31 18-12-56

YanielCarreno avatar Jun 01 '22 13:06 YanielCarreno

You should check at the sensor definition then. It seems like a default value not being set correctly (as instead of infinity you have a x m set as max distance or something).

Compare it to the kinetic version. There should be little difference in the xacro files between versions . And the sensors backend shouldn't have had major changes either.

  • check in your drone xacro what are the default values for this sensor, if nothing, go up to the sensor xacro, if still no luck check the cpp code of the sensor. Side note: I guess you know that most sonar like sensors on gazebo are experimental and not really realistic right? In this simulator they can easily be replaced by multilayer laserscans without much issue. If really you can't find the source.

my-name-is-D avatar Jun 01 '22 13:06 my-name-is-D