Closed Schneegans closed 1 year ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
plugins/csp-lod-bodies/src/MinMaxPyramid.hpp | 0 | 1 | 0.0% | ||
plugins/csp-lod-bodies/src/RenderDataDEM.cpp | 0 | 1 | 0.0% | ||
plugins/csp-lod-bodies/src/TileBounds.cpp | 0 | 1 | 0.0% | ||
plugins/csp-lod-bodies/src/PlanetShader.cpp | 0 | 2 | 0.0% | ||
plugins/csp-lod-bodies/src/Tile.hpp | 0 | 2 | 0.0% | ||
plugins/csp-lod-bodies/src/TerrainShader.cpp | 0 | 4 | 0.0% | ||
plugins/csp-lod-bodies/src/TileBase.cpp | 0 | 4 | 0.0% | ||
plugins/csp-lod-bodies/src/TileNode.cpp | 0 | 4 | 0.0% | ||
plugins/csp-lod-bodies/src/TileTextureArray.hpp | 0 | 5 | 0.0% | ||
plugins/csp-lod-bodies/src/VistaPlanet.cpp | 0 | 6 | 0.0% | ||
<!-- | Total: | 0 | 243 | 0.0% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
plugins/csp-lod-bodies/src/Plugin.cpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/RenderDataDEM.cpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/TileBase.hpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/Tile.hpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/TileNode.cpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/TileRenderer.hpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/TileSourceWebMapService.cpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/TileTextureArray.hpp | 1 | 0% | ||
plugins/csp-lod-bodies/src/LodBody.cpp | 2 | 0% | ||
plugins/csp-lod-bodies/src/MinMaxPyramid.cpp | 2 | 0% | ||
<!-- | Total: | 30 | --> |
Totals | |
---|---|
Change from base Build 4103295433: | 0.06% |
Covered Lines: | 515 |
Relevant Lines: | 16567 |
Thanks a lot for this review! I hope that I addressed all of your concerns :smile:. Do you have any feedback regarding the chosen default values?
Thanks a lot for this review! I hope that I addressed all of your concerns ๐. Do you have any feedback regarding the chosen default values?
One last comment remains. Otherwise it works great. I tried to break it with everything I could throw at it, but with the default configuration I wasn't able to find any bugs or holes in the ground. Only when I manually turned the LOD factor below 10 I was able to create situations where the planet surface had gaps between tiles, but at that point the application became useless anyways.
This is a great improvement. The render times are amazing!
This pull request tries to optimize the rendering performance of the
csp-lod-bodies
plugin. For this, it does the following:However, it does not:
The main advantages are:
The main drawbacks of this approach are:
Other notes:
MinMaxPyramid
is not really required anymore. However, we may need this in future for the tesselation shaders so I left it in the code. Yet I am not sure whether it's still working 100% as I had to replace many hard-coded numbers...