Skip to content

Beluga - #267

Open
Timple wants to merge 3 commits into
RoboStack:mainfrom
nobleo:feature/beluga
Open

Beluga#267
Timple wants to merge 3 commits into
RoboStack:mainfrom
nobleo:feature/beluga

Conversation

@Timple

@Timple Timple commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Timple

Timple commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, I'm gonna need help here. No idea why eigen isn't picked up as dependency...

@Tobias-Fischer

Copy link
Copy Markdown
Contributor

You need ros-jazzy-sophus.patch with

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0ea5b90..cc9a35b 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -54,7 +54,7 @@ endif()
 # defined by a parent project including Sophus via `add_subdirectory`.)
 
 if(NOT TARGET Eigen3::Eigen)
-  find_package(Eigen3 3.3.0 REQUIRED)
+  find_package(Eigen3 REQUIRED)
 endif()
 
 if(NOT TARGET fmt::fmt)

(don't have write access to your fork)

@Timple

Timple commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

Thank you!

Would you mind reviewing this then? If that's indeed the issue: conda-forge/sophus-feedstock#5

I didn't expect sophus could be patched here as it's in the conda-forge channel.

@Tobias-Fischer

Copy link
Copy Markdown
Contributor

The better way is to use the conda-forge version of sophus via a dummy package, once eigen there is updated.

@Timple

Timple commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

The better way is to use the conda-forge version of sophus via a dummy package, once eigen there is updated.

I need a little bit of explanation here. I like to use the conda-forge version of sophus. But why do we need a dummy package for that? What's the mechanism there.

@Timple

Timple commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

I'm going to start this one all-over again with the new sophus

@Timple

Timple commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

I'm going to need some help now. CI fails on:

 ╰─────────────────── (took 4 seconds)
Error:   × Failed to resolve dependencies: Cannot solve the request because of: ros-
  │ jazzy-ament-cmake-ros * cannot be installed because there are no viable
  │ options:
  │ └─ ros-jazzy-ament-cmake-ros 0.12.0 | 0.12.0 | 0.12.0 | 0.12.0 | 0.12.0 |
  │ 0.12.0 | 0.12.0 | 0.12.0 | 0.12.0 | 0.12.0 would require
  │    └─ ros2-distro-mutex ==0.5 jazzy, for which no candidates were found.

And I fail to see how this is related to this PR.

Locally these two commands work:

pixi run -v vinca
pixi run rattler-build build --recipe ./recipes/ros-jazzy-beluga/recipe.yaml -m ./conda_build_config.yaml -c conda-forge -c robostack-jazzy

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 this pull request may close these issues.

2 participants