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

3D view - completely flat MultiPolygonZ geometries don't show color correctly #52832

Open
2 tasks
andreasneumann opened this issue Apr 22, 2023 · 1 comment · May be fixed by #60895
Open
2 tasks

3D view - completely flat MultiPolygonZ geometries don't show color correctly #52832

andreasneumann opened this issue Apr 22, 2023 · 1 comment · May be fixed by #60895
Labels
3D Relates to QGIS' 3D engine or rendering Bug Either a bug report, or a bug fix. Let's hope for the latter!

Comments

@andreasneumann
Copy link
Member

andreasneumann commented Apr 22, 2023

What is the bug or the crash?

When a surface (e.g. triangle) is completely flat in a MultiPolygonZ geometry, the surface is rendered white instead of the brownish color that is assigned to roof surfaces.

image

Steps to reproduce the issue

  1. Load project flat_rendering_issue.zip
  2. Open 3D view
  3. The expected rendering would be that all roofs (including flat ones) are rendered in the same brownish color

image

As soon as you edit the z-value of one of the flat surfaces slightly (a mm is enough), then the color is correctly applied.

image

image

image

Versions

<style type="text/css"> p, li { white-space: pre-wrap; } </style>
QGIS version 3.31.0-Master QGIS code revision 0ce567b
Qt version 5.15.3
Python version 3.10.6
GDAL/OGR version 3.4.1
PROJ version 8.2.1
EPSG Registry database version v10.041 (2021-12-03)
GEOS version 3.10.2-CAPI-1.16.0
SQLite version 3.37.2
PDAL version 2.3.0
PostgreSQL client version unknown
SpatiaLite version 5.0.1
QWT version 6.1.4
QScintilla2 version 2.11.6
OS version Ubuntu 22.04.2 LTS
       
Active Python plugins
changeDataSource 3.1
solocator 1.6.0
pg_raster_import 3.2.0
trackable_project_files v1.2.2
linking_relation_editor v1.1.9
QgisModelBaker v7.5.0
AtlasExportFunction 1.1
processing 2.12.99
db_manager 0.1.20
QGIS version 3.31.0-Master QGIS code revision [0ce567b](https://github.com/qgis/QGIS/commit/0ce567bd306) Qt version 5.15.3 Python version 3.10.6 GDAL/OGR version 3.4.1 PROJ version 8.2.1 EPSG Registry database version v10.041 (2021-12-03) GEOS version 3.10.2-CAPI-1.16.0 SQLite version 3.37.2 PDAL version 2.3.0 PostgreSQL client version unknown SpatiaLite version 5.0.1 QWT version 6.1.4 QScintilla2 version 2.11.6 OS version Ubuntu 22.04.2 LTS

Active Python plugins
changeDataSource
3.1
solocator
1.6.0
pg_raster_import
3.2.0
trackable_project_files
v1.2.2
linking_relation_editor
v1.1.9
QgisModelBaker
v7.5.0
AtlasExportFunction
1.1
processing
2.12.99
db_manager
0.1.20

Supported QGIS version

  • I'm running a supported QGIS version according to the roadmap.

New profile

  • I tried with a new QGIS profile

Additional context

No response

@andreasneumann andreasneumann added Bug Either a bug report, or a bug fix. Let's hope for the latter! 3D Relates to QGIS' 3D engine or rendering labels Apr 22, 2023
@uclaros uclaros linked a pull request Mar 7, 2025 that will close this issue
@uclaros
Copy link
Contributor

uclaros commented Mar 7, 2025

@andreasneumann see the linked PR for a fix for this. Also note that "invert normals (experimental)" needs to be checked for the attached datasets to be properly rendered, as their polygons that face up have their vertices defined in a clockwise order.

We normally render clockwise polygons as facing down and counter clockwise as facing up, but this behavior can be flipped with the invert normals checkbox. We should probably communicate this to the user and make the cryptic culling mode, Add back faces and Invert normals a little more user friendly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3D Relates to QGIS' 3D engine or rendering Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants