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

Merge gz-rendering8 ➡️ main #934

Merged
merged 45 commits into from
Oct 10, 2023
Merged

Merge gz-rendering8 ➡️ main #934

merged 45 commits into from
Oct 10, 2023

Conversation

azeey
Copy link
Contributor

@azeey azeey commented Oct 10, 2023

➡️ Forward port

Port gz-rendering8 ➡️ main

Branch comparison: main...gz-rendering8

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

ahcorde and others added 30 commits June 12, 2023 17:48
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Merge gz-rendering7 ➡️  gz-rendering8
Signed-off-by: Addisu Z. Taddese <[email protected]>
* fix node indexing

---------

Signed-off-by: Ian Chen <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
Co-authored-by: Michael Carroll <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Martin Pecka <[email protected]>
Co-authored-by: Ian Chen <[email protected]>
Co-authored-by: Michael Carroll <[email protected]>
Sets the ogre2 target INSTALL_RPATH to a copy of
CMAKE_INSTALL_RPATH with OGRE_LIBRARY_DIRS
appended, instead of just the latter path. This
helps with packaging on macOS.

Signed-off-by: Steve Peters <[email protected]>
Sets the ogre2 target INSTALL_RPATH to a copy of
CMAKE_INSTALL_RPATH with OGRE_LIBRARY_DIRS
appended, instead of just the latter path. This
helps with packaging on macOS.

Signed-off-by: Steve Peters <[email protected]>
* Fix FOV calculation, projected texture orientation, crash when heighmaps are present

Signed-off-by: Ian Chen <[email protected]>
Co-authored-by: Addisu Z. Taddese <[email protected]>
* remove gazebo scene viewer example

---------

Signed-off-by: Ian Chen <[email protected]>
* Calculate camera intrinsics : Refactor #700 (#755)


Signed-off-by: Alejandro Hernández Cordero <[email protected]>

---------

Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Aditya Pande <[email protected]>
Merge ign-rendering6 ➡️  gz-rendering7
Signed-off-by: Addisu Z. Taddese <[email protected]>
iche033 and others added 15 commits September 27, 2023 08:01
Signed-off-by: Addisu Z. Taddese <[email protected]>
Updated actions/checkout to v4

Signed-off-by: mosfet80 <[email protected]>
* Extend ogre 1.x shaders to support constants and textures
* add waves integration test

---------

Signed-off-by: Ian Chen <[email protected]>
@azeey azeey requested a review from iche033 as a code owner October 10, 2023 20:20
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #934 (8fa62b1) into main (30ec9d4) will increase coverage by 0.40%.
The diff coverage is 64.70%.

❗ Current head 8fa62b1 differs from pull request most recent head 3637c76. Consider uploading reports for the commit 3637c76 to get more accurate results

@@            Coverage Diff             @@
##             main     #934      +/-   ##
==========================================
+ Coverage   75.26%   75.66%   +0.40%     
==========================================
  Files         176      177       +1     
  Lines       16853    16884      +31     
==========================================
+ Hits        12685    12776      +91     
+ Misses       4168     4108      -60     
Files Coverage Δ
include/gz/rendering/base/BaseNativeWindow.hh 0.00% <ø> (ø)
ogre2/src/Ogre2Material.cc 84.76% <100.00%> (+8.68%) ⬆️
ogre2/src/Ogre2ParticleEmitter.cc 78.01% <100.00%> (-0.57%) ⬇️
ogre2/src/Ogre2RenderEngine.cc 68.18% <ø> (ø)
include/gz/rendering/base/BaseAxisVisual.hh 62.71% <0.00%> (ø)
ogre2/src/Ogre2Scene.cc 78.53% <83.33%> (+0.03%) ⬆️
ogre2/src/Ogre2GlobalIlluminationCiVct.cc 0.00% <0.00%> (ø)
src/GlobalIlluminationBase.cc 0.00% <0.00%> (ø)
src/GlobalIlluminationVct.cc 0.00% <0.00%> (ø)
src/NativeWindow.cc 0.00% <0.00%> (ø)
... and 5 more

... and 1 file with indirect coverage changes

@azeey azeey merged commit 9382513 into main Oct 10, 2023
6 of 7 checks passed
@azeey azeey deleted the azeey/8_to_main branch October 10, 2023 22:37
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.

7 participants