-
Notifications
You must be signed in to change notification settings - Fork 22
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
Nix #268
Conversation
There was a problem hiding this 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)'
Hello @LudovicDeMatteis @nim65s, I think to get this merged we'd only need to update the Changelog (if you are indeed ready to merge). |
861d8cf
to
a4df6e7
Compare
@ManifoldFR I rebased on devel and added the Nix support to the Changelog |
There was a problem hiding this 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 😃
Oops @LudovicDeMatteis it seems the Nix step failed |
@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. |
Yes, we need to
This will fix nix CI, but probably break all other ones. Let's try :) |
Ok, this is overkill, sorry. Let's keep this for later, and allow some failing tests for now. |
We can make a new pinocchio release if it help. |
If you have some time for new releases, yes, eigenpy + coal + pinocchio would help a lot, thanks :D |
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).