Skip to content

[Device] Make class derive from std::enable_shared_from_this.#191

Merged
florent-lamiraux merged 6 commits intohumanoid-path-planner:develfrom
florent-lamiraux:devel
Jan 22, 2025
Merged

[Device] Make class derive from std::enable_shared_from_this.#191
florent-lamiraux merged 6 commits intohumanoid-path-planner:develfrom
florent-lamiraux:devel

Conversation

@florent-lamiraux
Copy link
Contributor

This modification fixes a bug in the way hpp::python handles weak pointers.
See https://stackoverflow.com/questions/8233252/boostpython-and-weak-ptr-stuff-disappearing for details about the issue and the recommended fix.

pre-commit-ci bot and others added 3 commits January 20, 2025 17:13
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.9.1 → v0.9.2](astral-sh/ruff-pre-commit@v0.9.1...v0.9.2)
- [github.com/pre-commit/mirrors-clang-format: v19.1.6 → v19.1.7](pre-commit/mirrors-clang-format@v19.1.6...v19.1.7)
  This modification fixes a bug in the way hpp::python handles weak pointers.
  See https://stackoverflow.com/questions/8233252/boostpython-and-weak-ptr-stuff-disappearing
  for details about the issue and the recommended fix.
@florent-lamiraux
Copy link
Contributor Author

Now requires humanoid-path-planner/hpp-pinocchio#225.

@florent-lamiraux florent-lamiraux merged commit b8c98bd into humanoid-path-planner:devel Jan 22, 2025
4 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.

2 participants