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

Build Dave | Error #272

Closed
Z8MAN8 opened this issue Jun 21, 2023 · 7 comments
Closed

Build Dave | Error #272

Z8MAN8 opened this issue Jun 21, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@Z8MAN8
Copy link

Z8MAN8 commented Jun 21, 2023

Hello, when I build a project based on the documentation, I encounter the following problem:
-- +++ processing catkin package: 'nps_uw_multibeam_sonar'
-- ==> add_subdirectory(nps_uw_multibeam_sonar)
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- Could NOT find acoustic_msgs (missing: acoustic_msgs_DIR)
-- Could not find the required component 'acoustic_msgs'. The following CMake error indicates that you either need to install the package with the same name or change your environment so that it can be found.
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkinConfig.cmake:83 (find_package):
Could not find a package configuration file provided by "acoustic_msgs"
with any of the following names:

acoustic_msgsConfig.cmake
acoustic_msgs-config.cmake

Add the installation prefix of "acoustic_msgs" to CMAKE_PREFIX_PATH or set
"acoustic_msgs_DIR" to a directory containing one of the above files. If
"acoustic_msgs" provides a separate development package or SDK, be sure it
has been installed.
Call Stack (most recent call first):
nps_uw_multibeam_sonar/CMakeLists.txt:17 (find_package)

-- Configuring incomplete, errors occurred!
See also "/home/csc/uuv_ws/build/CMakeFiles/CMakeOutput.log".
See also "/home/csc/uuv_ws/build/CMakeFiles/CMakeError.log".
Invoking "cmake" failed

I can't solve this problem myself, can you help me?Thank you

@mabelzhang
Copy link
Contributor

mabelzhang commented Jun 26, 2023

Hi, this is an upstream error. We encountered this in ROS 2 Humble in another project. Our solution is to revert to an earlier commit to bypass that error https://github.com/osrf/mbari_wec/pull/51/files

Looks like you're using ROS 1 since it's catkin. There might be other upstream dependencies using actuator_msgs... We traced the following for why it didn't work for Humble. For other distros, or ROS 1, it might be something similar but not exactly the same.

The cause is that ros_gz_bridge has a new dependency on actuator_msgs, which had been released into Humble https://github.com/ros/rosdistro/pulls?q=is%3Apr+actuator_msgs+ But at the time we encountered the error, Humble didn't have a sync yet, so there was no binary package like ros-humble-actuator-msgs on Ubuntu 22.04.

There has been some releases of ros_gz on rosdistro since we saw the problem on May 2. e.g. this one into Humble on May 23 ros/rosdistro#37346

A Humble sync has come out on May 30 https://discourse.ros.org/t/preparing-for-humble-sync-2023-05-30/31622/2

So maybe it's better now? I haven't checked recently.

@mabelzhang
Copy link
Contributor

mabelzhang commented Jun 26, 2023

Ah, sorry, in the above comment, I was confused by the name acoustic_msgs. There is an upstream message called acoustic_msgs, but the one used in DAVE is a completely different package from APL https://github.com/Field-Robotics-Lab/nps_uw_multibeam_sonar/blob/50c34356513d6d0827eb5ad6f5cc691caf06d7ab/package.xml#L23

This PR should fix your error #269 . You might need to replace acoustic_msgs with marine_acoustic_msgs too, as there seems to have been a rename https://github.com/apl-ocean-engineering/marine_msgs/tree/main/marine_acoustic_msgs

Let us know if it fixes it for you, so that we can look into merging that PR and closing this issue.
Thanks!

@CCLings
Copy link

CCLings commented Aug 15, 2023

In the folder uuv_ws/src, this project cloned acoustic_msgs package from https://github.com/apl-ocean-engineering/marine_msgs/tree/main/marine_acoustic_msgs to the folder hydrographic_msgs. But unfortunately, the package has been updated, NOT compatible. If you just want to run codes, the simplest way is the rollback of the downloaded package.

cd uuv_ws/src/hydrographic_msgs
git checkout 5344060071d0a72857a9efb558cc1f2d7c0052c6

@mabelzhang
Copy link
Contributor

Yeah, #269 updates that dependency.
I haven't had time to test out that PR, though I believe it most likely works because it comes from the authors of hydrographic_msgs. If anyone has time to test it and post an approval there, and we have multiple approvals, I'll merge it.

@huangbintao188
Copy link

Could you tell me how to solve this? thank you.

@woensug-choi
Copy link
Collaborator

@huangbintao188 working on this. I will ping you back when I get the update uploaded.

@woensug-choi
Copy link
Collaborator

#269 and Field-Robotics-Lab/nps_uw_multibeam_sonar#55 would have solved this. If the problem persists. let me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants