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

Cherry-pick LensFlare api changes to 11 #2974

Merged
merged 4 commits into from
Apr 20, 2021

Conversation

scpeters
Copy link
Member

@scpeters scpeters commented Apr 17, 2021

Cherry-pick the following pull requests forward to gazebo11. Use rebase and merge.

@scpeters scpeters requested a review from iche033 April 17, 2021 00:55
* Add LensFlarePrivate setter methods and move
  LensFlareCompositorListener defintion to header

* Modified LensFlareCompositorListener members to follow PIMPL

* Reduce line length and fix whitespace

Signed-off-by: Steve Peters <scpeters@openrobotics.org>

Co-authored-by: kbjeppes <kaden.b.jeppesen@nasa.gov>
Co-authored-by: Steve Peters <scpeters@openrobotics.org>
scpeters and others added 3 commits April 19, 2021 16:44
This is an adjustment to API added in gazebosim#2965, which is allowed
because the API has not yet been released.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Martin Pecka <peckama2@fel.cvut.cz>
* Remove modifications to the shadow technique done in the
DepthCamera, this is not necessary and breaks lighting for all
other cameras that will render after it in the same frame

* Add call to setShadowsEnabled for the normals viewport as well

Co-authored-by: Steve Peters <scpeters@openrobotics.org>
@scpeters scpeters force-pushed the pick_lensflare_api branch from 8e4c464 to f04fc15 Compare April 19, 2021 23:45
@scpeters
Copy link
Member Author

rebased and added #2975 and #2907

@scpeters scpeters merged commit 594d807 into gazebosim:gazebo11 Apr 20, 2021
@scpeters scpeters deleted the pick_lensflare_api branch April 20, 2021 16:53
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.

5 participants