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

6 -> main #592

Merged
merged 5 commits into from
Mar 22, 2022
Merged

6 -> main #592

merged 5 commits into from
Mar 22, 2022

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Mar 22, 2022

➡️ Forward port

Port ign-rendering6 to main

Branch comparison: main...ign-rendering6

changes from #575 and #579 are not merged forward. They are replaced by #578 which contains a better fix for main

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

iche033 and others added 5 commits March 4, 2022 16:52
…575)

* fix shaders when there is lidar in the scene

Signed-off-by: Ian Chen <[email protected]>

* style

Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
…#579)

This PR fixes objects with shader material when there is a thermal camera or a segmentation camera in the scene. These sensor all perform material switching during a render update, and the shader materials should now be correctly restored. Also updated test.

Signed-off-by: Ian Chen <[email protected]>
* Use libogre-next-dev on jammy

Specify different package names for ogre2.2
in distro-specific packages-*.apt files.

* Find OGRE-Next with plain cmake

Patch adapted from gazebo-release/gz-rendering6-release#8

Signed-off-by: Steve Peters <[email protected]>
Co-authored-by: Louise Poubel <[email protected]>
@iche033
Copy link
Contributor Author

iche033 commented Mar 22, 2022

Comments associated with merged forward commits should already be addressed :)

@github-actions github-actions bot added the 🌱 garden Ignition Garden label Mar 22, 2022
@codecov
Copy link

codecov bot commented Mar 22, 2022

Codecov Report

Merging #592 (bbb2fc4) into main (6f5faaf) will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #592      +/-   ##
==========================================
+ Coverage   53.43%   53.55%   +0.11%     
==========================================
  Files         214      214              
  Lines       21304    21304              
==========================================
+ Hits        11384    11409      +25     
+ Misses       9920     9895      -25     
Impacted Files Coverage Δ
ogre2/src/Ogre2GpuRays.cc 91.95% <ø> (+1.06%) ⬆️
ogre2/src/Ogre2ThermalCamera.cc 82.13% <ø> (+1.74%) ⬆️
ogre2/src/Ogre2SegmentationCamera.cc 75.40% <0.00%> (+0.53%) ⬆️
include/ignition/rendering/base/BaseCamera.hh 65.68% <0.00%> (+0.73%) ⬆️
ogre2/src/Ogre2SegmentationMaterialSwitcher.cc 64.73% <0.00%> (+4.04%) ⬆️

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 6f5faaf...bbb2fc4. Read the comment docs.

@iche033 iche033 merged commit f3c4003 into main Mar 22, 2022
@iche033 iche033 deleted the merge_6_7_20220321 branch March 22, 2022 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants