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

Fix SDF plugin variables for MulticopterMotorModel #1981

Open
wants to merge 6 commits into
base: gz-sim7
Choose a base branch
from

Conversation

bperseghetti
Copy link
Collaborator

@bperseghetti bperseghetti commented May 2, 2023

🦟 Bug fix

Fixes #1980

Summary

Updated the MulticopterMotorModel plugin to use unified SDF variable
formatting. Included a deprecation warning for users that the old variables will be
removed in harmonic.

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

@github-actions github-actions bot added the 🌱 garden Ignition Garden label May 2, 2023
@osrf-triage osrf-triage added this to Inbox in Core development May 2, 2023
@bperseghetti bperseghetti force-pushed the pr-mmm-sdf-fix branch 2 times, most recently from ec77ecd to 7c61f1b Compare May 2, 2023 04:14
@azeey
Copy link
Contributor

azeey commented May 2, 2023

FWIW, the parameter names were directly taken from https://github.com/ethz-asl/rotors_simulator/blob/master/rotors_gazebo_plugins/src/gazebo_motor_model.cpp to make it easier for users to transition from RotorS based models.

@azeey azeey requested a review from ahcorde May 8, 2023 18:46
@azeey azeey moved this from Inbox to In review in Core development May 8, 2023
@azeey azeey added the beta Targeting beta release of upcoming collection label Jul 31, 2023
@azeey
Copy link
Contributor

azeey commented Aug 21, 2023

The plan is to make backward compatible changes in ign-gazebo6 and gz-sim7 without warnings (maybe with a todo to enable warnings) and then enable warnings in main (Harmonic). And then remove the old parameters in Gazebo-I

Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

pending comments

@azeey azeey removed the beta Targeting beta release of upcoming collection label Aug 29, 2023
@azeey
Copy link
Contributor

azeey commented Aug 29, 2023

Removing beta from this as well per our conversation with Ben.

Updated the Multicopter Motor Model plugin to use unified SDF variable
formatting. Included a deprecation warning for users that the old variables will be
removed in harmonic.

Signed-off-by: Benjamin Perseghetti <[email protected]>
Signed-off-by: Benjamin Perseghetti <[email protected]>
Signed-off-by: Benjamin Perseghetti <[email protected]>
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

@azeey any other comments ?

Signed-off-by: Benjamin Perseghetti <[email protected]>
@azeey azeey added the beta Targeting beta release of upcoming collection label Jul 29, 2024
@azeey
Copy link
Contributor

azeey commented Aug 6, 2024

@bperseghetti can you merge from gz-sim7 to hopefully get green CI? Also, I suggest we keep the old plugin variables for one of the tests to make sure this is still backward compatible.

@azeey azeey removed the beta Targeting beta release of upcoming collection label Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

MulticopterMotorModel uses improper SDF formatting.
3 participants