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

Pinocchio v3 compatibility #1253

Merged
merged 9 commits into from
May 31, 2024
Merged

Pinocchio v3 compatibility #1253

merged 9 commits into from
May 31, 2024

Conversation

nim65s
Copy link
Collaborator

@nim65s nim65s commented May 28, 2024

No description provided.

nim65s and others added 7 commits May 19, 2024 10:21
ResidualModelFactory was allocating the content of a `std::shared_ptr`
manually (with a new).
On GNU/Linux with G++, the standard library use free to deallocate this
memory.
This is an undefined behavior that cause random error on
at least test_residual unit test.
This fix probably hide a bigger issue in the Python binding.
Copy link
Member

@cmastalli cmastalli left a comment

Choose a reason for hiding this comment

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

Thanks for adding compatibility with Pinocchio 3, @nim65s!

Let us sort out the breaking of Python API. The problem might arise from recent changes in eigenpy.

unittest/bindings/factory.py Show resolved Hide resolved
@nim65s
Copy link
Collaborator Author

nim65s commented May 28, 2024

As you can see in the commit history, @jorisv is doing most of the work, and deserve the credits :)
I just opened this draft so that we can more easily talk about those changes for now.

@nim65s nim65s changed the title Pin3 Pinocchio v3 compatibility May 28, 2024
@nim65s
Copy link
Collaborator Author

nim65s commented May 28, 2024

NB: there is another blocking pointsthan what we can see here. It is being worked on, in private spaces. @LudovicDeMatteis will be able to provide a green light when everything will be ready enough for him.

update: this was stack-of-tasks/pinocchio#2247, and is not related to crocoddyl. Sorry for the false alarm.

@nim65s nim65s marked this pull request as ready for review May 30, 2024 13:52
@nim65s nim65s merged commit 9dccb6a into devel May 31, 2024
10 of 17 checks passed
nim65s added a commit to nim65s/robotpkg that referenced this pull request May 31, 2024
Changed in v2.1.0:

* Updated black + isort + flake8 to ruff in loco-3d/crocoddyl#1256
* Exported version for Python in loco-3d/crocoddyl#1254
* Added pinocchio 3 preliminary support in loco-3d/crocoddyl#1253
* Updated CMake packaging in loco-3d/crocoddyl#1249
* Fixed ruff reported error in loco-3d/crocoddyl#1248
* Fixed yapf reported errors in loco-3d/crocoddyl#1238
* Tested Python stubs in Conda CI in loco-3d/crocoddyl#1228
* Fixed Rviz display in loco-3d/crocoddyl#1227
* Improved CI, updated cmake and fixed launch file in loco-3d/crocoddyl#1220
* Introduced a Rviz display in loco-3d/crocoddyl#1216
* Enabled display of thrust and simplied displayers code in loco-3d/crocoddyl#1215
* Introduced floating base thruster actuation model in loco-3d/crocoddyl#1213
* Fixed quadruped and biped examples in loco-3d/crocoddyl#1208
* Fixed terminal computation in Python models in loco-3d/crocoddyl#1204
* Fixed handling of unbounded values for `ActivationBounds` in loco-3d/crocoddyl#1191
@cmastalli cmastalli deleted the pin3 branch June 6, 2024 06:02
nim65s added a commit to nim65s/robotpkg that referenced this pull request Jun 20, 2024
Changed in v2.1.0:

* Updated black + isort + flake8 to ruff in loco-3d/crocoddyl#1256
* Exported version for Python in loco-3d/crocoddyl#1254
* Added pinocchio 3 preliminary support in loco-3d/crocoddyl#1253
* Updated CMake packaging in loco-3d/crocoddyl#1249
* Fixed ruff reported error in loco-3d/crocoddyl#1248
* Fixed yapf reported errors in loco-3d/crocoddyl#1238
* Tested Python stubs in Conda CI in loco-3d/crocoddyl#1228
* Fixed Rviz display in loco-3d/crocoddyl#1227
* Improved CI, updated cmake and fixed launch file in loco-3d/crocoddyl#1220
* Introduced a Rviz display in loco-3d/crocoddyl#1216
* Enabled display of thrust and simplied displayers code in loco-3d/crocoddyl#1215
* Introduced floating base thruster actuation model in loco-3d/crocoddyl#1213
* Fixed quadruped and biped examples in loco-3d/crocoddyl#1208
* Fixed terminal computation in Python models in loco-3d/crocoddyl#1204
* Fixed handling of unbounded values for `ActivationBounds` in loco-3d/crocoddyl#1191
nim65s added a commit to nim65s/robotpkg that referenced this pull request Jul 1, 2024
Changed in v2.1.0:

* Updated black + isort + flake8 to ruff in loco-3d/crocoddyl#1256
* Exported version for Python in loco-3d/crocoddyl#1254
* Added pinocchio 3 preliminary support in loco-3d/crocoddyl#1253
* Updated CMake packaging in loco-3d/crocoddyl#1249
* Fixed ruff reported error in loco-3d/crocoddyl#1248
* Fixed yapf reported errors in loco-3d/crocoddyl#1238
* Tested Python stubs in Conda CI in loco-3d/crocoddyl#1228
* Fixed Rviz display in loco-3d/crocoddyl#1227
* Improved CI, updated cmake and fixed launch file in loco-3d/crocoddyl#1220
* Introduced a Rviz display in loco-3d/crocoddyl#1216
* Enabled display of thrust and simplied displayers code in loco-3d/crocoddyl#1215
* Introduced floating base thruster actuation model in loco-3d/crocoddyl#1213
* Fixed quadruped and biped examples in loco-3d/crocoddyl#1208
* Fixed terminal computation in Python models in loco-3d/crocoddyl#1204
* Fixed handling of unbounded values for `ActivationBounds` in loco-3d/crocoddyl#1191
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.

3 participants