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

Nix #268

Merged
merged 7 commits into from
Feb 11, 2025
Merged

Nix #268

merged 7 commits into from
Feb 11, 2025

Conversation

LudovicDeMatteis
Copy link
Collaborator

Same as stack-of-tasks/eigenpy#498

For some reason (looks like a pinocchio version issue), the build fails on devel for now (but does pass on main).

nim65s
nim65s previously approved these changes Jan 27, 2025
Copy link
Member

@nim65s nim65s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM. The bug in pinocchio version could be treated in another PR I think, but it would also be fine here.

aligator> [ 74%] Linking CXX executable ../bin/aligator-example-clqr
aligator> /nix/store/a1cr17kwgakznfw4pp8paw8yv189ikk5-binutils-2.43.1/bin/ld: ../lib/libaligator.so.0.10.0: undefined reference to `pinocchio::DataTpl<double, 0, pinocchio::JointCollectionDefaultTpl>::TangentVectorType const& pinocchio::aba<double, 0, pinocchio::JointCollectionDefaultTpl, Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> >, Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> >, Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> > >(pinocchio::ModelTpl<double, 0, pinocchio::JointCollectionDefaultTpl> const&, pinocchio::DataTpl<double, 0, pinocchio::JointCollectionDefaultTpl>&, Eigen::MatrixBase<Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> > > const&, Eigen::MatrixBase<Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> > > const&, Eigen::MatrixBase<Eigen::Ref<Eigen::Matrix<double, -1, 1, 0, -1, 1> const, 0, Eigen::InnerStride<1> > > const&, pinocchio::Convention)'

@ManifoldFR
Copy link
Member

Hello @LudovicDeMatteis @nim65s, I think to get this merged we'd only need to update the Changelog (if you are indeed ready to merge).

@LudovicDeMatteis
Copy link
Collaborator Author

@ManifoldFR I rebased on devel and added the Nix support to the Changelog

ManifoldFR
ManifoldFR previously approved these changes Feb 10, 2025
Copy link
Member

@ManifoldFR ManifoldFR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot as always, gentlemen 😃

@ManifoldFR
Copy link
Member

Oops @LudovicDeMatteis it seems the Nix step failed

@LudovicDeMatteis
Copy link
Collaborator Author

@ManifoldFR It is still due to the Pinocchio's ABA template instantiation error, that we need to either wait for the next Pinocchio Release to merge or include a patch in the nix flake until then.

@nim65s
Copy link
Member

nim65s commented Feb 10, 2025

@nim65s
Copy link
Member

nim65s commented Feb 10, 2025

Ok, this is overkill, sorry. Let's keep this for later, and allow some failing tests for now.

@jorisv
Copy link
Contributor

jorisv commented Feb 10, 2025

We can make a new pinocchio release if it help.

@nim65s
Copy link
Member

nim65s commented Feb 10, 2025

If you have some time for new releases, yes, eigenpy + coal + pinocchio would help a lot, thanks :D

@ManifoldFR ManifoldFR enabled auto-merge (rebase) February 11, 2025 00:31
@ManifoldFR ManifoldFR merged commit ffd71a4 into Simple-Robotics:devel Feb 11, 2025
17 checks passed
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.

4 participants