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

Don't clone visuals with an ignore flag #442

Closed
wants to merge 2 commits into from

Conversation

adlarkin
Copy link
Contributor

Signed-off-by: Ashton Larkin [email protected]

Summary

While working on gazebosim/gz-sim#1013, I noticed that there are times where certain visuals shouldn't be cloned. I've added support for setting a boolean UserData flag with a key of skip-visual-clone on a visual. If a visual has UserData with this key, it will not be cloned.

For an example of how this is useful, see the second half of gazebosim/gz-sim#1013 (comment).

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 to maintainers: Remember to use Squash-Merge

@adlarkin adlarkin requested a review from iche033 as a code owner September 29, 2021 23:02
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Sep 29, 2021
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #442 (dc6cdfd) into ign-rendering6 (34d1d0a) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           ign-rendering6     #442      +/-   ##
==================================================
+ Coverage           53.46%   53.48%   +0.02%     
==================================================
  Files                 192      192              
  Lines               19536    19545       +9     
==================================================
+ Hits                10444    10454      +10     
+ Misses               9092     9091       -1     
Impacted Files Coverage Δ
include/ignition/rendering/Visual.hh 100.00% <ø> (ø)
include/ignition/rendering/base/BaseVisual.hh 71.49% <100.00%> (+1.29%) ⬆️
...e/ignition/rendering/base/BaseGaussianNoisePass.hh 100.00% <0.00%> (+3.33%) ⬆️

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 34d1d0a...dc6cdfd. Read the comment docs.

@adlarkin adlarkin mentioned this pull request Sep 29, 2021
8 tasks
@adlarkin adlarkin changed the base branch from main to ign-rendering6 October 4, 2021 20:46
@adlarkin
Copy link
Contributor Author

I'm closing this PR because this functionality is now a part of ign-gazebo: see gazebosim/gz-sim#1103

@adlarkin adlarkin closed this Oct 13, 2021
@adlarkin adlarkin deleted the adlarkin/ignore_gui_only_visuals branch October 13, 2021 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants