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

Package request: ros-humble-slam-toolbox #133

Open
1 of 5 tasks
aptxlee opened this issue Feb 6, 2024 · 4 comments
Open
1 of 5 tasks

Package request: ros-humble-slam-toolbox #133

aptxlee opened this issue Feb 6, 2024 · 4 comments
Labels
windows wontfix This will not be worked on

Comments

@aptxlee
Copy link

aptxlee commented Feb 6, 2024

Package name

ros-humble-slam-toolbox

Operating System(s)

  • linux-64
  • linux-aarch64
  • osx-64
  • osx-arm64
  • win-64

Additional context (optional)

It is a package that can run in Windows binaries, and must be supported to perform full nav2 functions.

Links and references (optional)

No response

@Tobias-Fischer
Copy link
Contributor

This currently fails as csparse cannot be found. Note that the last version of suitesparse on Windows is very old, see conda-forge/suitesparse-feedstock#51 (but it seems feasible to build as that's what they do in cvxopt)

/cc @traversaro

@edvart-ros
Copy link

Bump! @traversaro is this doable?

@traversaro
Copy link
Member

As reported in conda-forge/suitesparse-feedstock#51 updating suitesparse on windows seems doable, however I do not have direct interest on doing this at the moment, so I can't work on this myself. If anyone wants to volunteer to work on this I would be happy to advise and help.

@traversaro
Copy link
Member

As reported in conda-forge/suitesparse-feedstock#51 updating suitesparse on windows seems doable, however I do not have direct interest on doing this at the moment, so I can't work on this myself. If anyone wants to volunteer to work on this I would be happy to advise and help.

Ok, actually there was no need to update suitesparse, the fix necessary to find suitesparse was just SteveMacenski/slam_toolbox#699 .

I looked a bit on the remaining failure, but it seems that the package requires several changes to build on Windows, see https://github.com/SteveMacenski/slam_toolbox/pull/440/files . However, to correctly support stack_size_to_use parameter, it is necessary to do some further changes, described in SteveMacenski/slam_toolbox#439 (comment) . At the moment I do not plan to work on this, but if anyone wants to work on it I will be happy to advice and/or help.

It also necessary to have compatibility with the latest ceres, but this is given by SteveMacenski/slam_toolbox#687 .

@Tobias-Fischer Tobias-Fischer added the wontfix This will not be worked on label Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
windows wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants