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 rotating cubes when placed by GUI tool while using the buoyancy plugin #1064

Merged
merged 5 commits into from
Sep 28, 2021

Conversation

arjo129
Copy link
Contributor

@arjo129 arjo129 commented Sep 28, 2021

🦟 Bug fix

Fixes #

Summary

Previously @aaronchongth discovered that the following behaviour takes place when adding shapes using the GUI tool while running the simulation

done.mp4

While this makes for great VFX, it is certainly not intended behavior. The root cause was eventually traced to the fact that when the gui tool is used new objects don't have a pose. The buoyancy plugin creates a pose component but does not wait for an update cycle. This PR moves the point at which the pose is read. Thus in the first preupdate cycle, the plugin will do nothing but enable the pose component.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • 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

Note

If you test this in the fortress demo world you may find the cubes sink really slowly. This is fine, as the cubes are really light given their volume and the plugin is configured to simulate air.

@arjo129 arjo129 changed the base branch from ign-gazebo5 to main September 28, 2021 09:29
@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Sep 28, 2021
@arjo129 arjo129 added 🏯 fortress Ignition Fortress and removed 🏢 edifice Ignition Edifice labels Sep 28, 2021
Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Arjo Chakravarty <[email protected]>
@arjo129 arjo129 changed the title Arjo/fix/buoyancyengine Fix rotating cubes when placed while using the buoyancy engine plugin Sep 28, 2021
@arjo129 arjo129 changed the title Fix rotating cubes when placed while using the buoyancy engine plugin Fix rotating cubes when placed by GUI tool while using the buoyancy plugin Sep 28, 2021
@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #1064 (a5eb1f2) into main (57f672d) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1064      +/-   ##
==========================================
- Coverage   64.08%   64.01%   -0.07%     
==========================================
  Files         255      255              
  Lines       19917    19919       +2     
==========================================
- Hits        12763    12752      -11     
- Misses       7154     7167      +13     
Impacted Files Coverage Δ
src/systems/buoyancy/Buoyancy.cc 82.53% <100.00%> (+0.18%) ⬆️
...int_position_controller/JointPositionController.cc 53.14% <0.00%> (-7.43%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57f672d...a5eb1f2. Read the comment docs.

@chapulina chapulina added beta Targeting beta release of upcoming collection bug Something isn't working labels Sep 28, 2021
Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina merged commit f8781ab into main Sep 28, 2021
@chapulina chapulina deleted the arjo/fix/buoyancyengine branch September 28, 2021 17:02
WilliamLewww pushed a commit to WilliamLewww/ign-gazebo that referenced this pull request Dec 7, 2021
…lugin (gazebosim#1064)

Signed-off-by: Arjo Chakravarty <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>

Co-authored-by: Louise Poubel <[email protected]>
Signed-off-by: William Lew <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Targeting beta release of upcoming collection bug Something isn't working 🏯 fortress Ignition Fortress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants