Add reading support for multi-overview VPC files#65384
Add reading support for multi-overview VPC files#65384uclaros wants to merge 6 commits intoqgis:masterfrom
Conversation
|
This pull request has been tagged for the changelog.
You can edit the description. Format available for credits
Thank you! |
🪟 Windows Qt6 buildsDownload Windows Qt6 builds of this PR for testing. 🍎 MacOS Qt6 buildsDownload MacOS Qt6 builds of this PR for testing. |
Tests failed for Qt 6 (ALL_BUT_PROVIDERS - fedora)One or more tests failed using the build from commit 0f70302 classified_render_overviewclassified_render_overviewTest failed at testOverviewRender at tests/src/python/test_qgspointcloudclassifiedrenderer.py:496 Rendered image did not match tests/testdata/control_images/pointcloudrenderer/expected_classified_render_overview/expected_classified_render_overview.png (found 2947 pixels different) The full test report (included comparison of rendered vs expected images) can be found here. Further documentation on the QGIS test infrastructure can be found in the Developer's Guide. |
Tests failed for Qt 6 (ALL_BUT_PROVIDERS - ubuntu)One or more tests failed using the build from commit 0f70302 classified_render_overviewclassified_render_overviewTest failed at testOverviewRender at tests/src/python/test_qgspointcloudclassifiedrenderer.py:496 Rendered image did not match tests/testdata/control_images/pointcloudrenderer/expected_classified_render_overview/expected_classified_render_overview.png (found 2947 pixels different) The full test report (included comparison of rendered vs expected images) can be found here. Further documentation on the QGIS test infrastructure can be found in the Developer's Guide. |
Description
Instead of parsing overview assets by their "overview" id, we now parse them based on their "overview" role.
VPC files may contain multiple overviews per asset with arbitrary ids.
All assets with "overview" role are now considered as overviews, loaded and rendered when zoomed out.
QgsPointCloudLayerandQgsVirtualPointCloudProvidernow have anoverviews()method returning a list of all overviews, instead of a single one.Support for generating such vpcs using
pdal_wrenchis pending: PDAL/wrench#86AI tool usage