Skip to content

python: suppress Pose3 deprecation warnings#682

Merged
scpeters merged 2 commits intogz-math7from
scpeters/suppress_pose3_python_deprecation_warnings
Jun 17, 2025
Merged

python: suppress Pose3 deprecation warnings#682
scpeters merged 2 commits intogz-math7from
scpeters/suppress_pose3_python_deprecation_warnings

Conversation

@scpeters
Copy link
Copy Markdown
Member

@scpeters scpeters commented Jun 17, 2025

🦟 Bug fix

Fixes compilation warnings on macOS, part of #60.

Summary

Several deprecated operators of the Pose3 class (see #60) are causing compilation warnings in our python bindings. This uses a more verbose way of specifying these operator bindings, while adding a console warning message, and suppressing the compiler warnings.

Build Status https://build.osrfoundation.org/job/gz_math-ci-gz-math7-homebrew-amd64/151/clang/

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Remove this if GenAI was not used.

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@github-actions github-actions bot added the 🎵 harmonic Gazebo Harmonic label Jun 17, 2025
@scpeters
Copy link
Copy Markdown
Member Author

@osrf-jenkins run tests please

@scpeters scpeters marked this pull request as ready for review June 17, 2025 03:03
@scpeters scpeters requested a review from Crola1702 June 17, 2025 03:04
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Jun 17, 2025
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@scpeters scpeters merged commit ad7e7cd into gz-math7 Jun 17, 2025
11 checks passed
@scpeters scpeters deleted the scpeters/suppress_pose3_python_deprecation_warnings branch June 17, 2025 16:57
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎵 harmonic Gazebo Harmonic

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants