Add some thread safety to Qgs3DMapSettings usage

CMakeLists.txt

@@ -1103,7 +1103,7 @@ if (WITH_CORE AND WITH_BINDINGS)
   include(SIPMacros)
 
   set(SIP_INCLUDES ${PYQT_SIP_DIR} ${CMAKE_SOURCE_DIR}/python)
-  set(SIP_CONCAT_PARTS 25)
+  set(SIP_CONCAT_PARTS 22)
 
   if (NOT BINDINGS_GLOBAL_INSTALL)
     set(Python_SITEARCH ${QGIS_DATA_DIR}/python)

python/3d/auto_generated/qgs3dmapsettings.sip.in

@@ -16,6 +16,12 @@ class Qgs3DMapSettings : QObject, QgsTemporalRangeObject
 {
 %Docstring(signature="appended")
 Definition of the world.
+
+.. warning::
+
+   Qgs3DMapSettings are a QObject subclass, and accordingly are not
+   safe for access across different threads. See Qgs3DRenderContext instead
+   for a safe snapshot of settings from Qgs3DMapSettings.
 %End
 
 %TypeHeaderCode
@@ -74,10 +80,15 @@ with scenes spanning hundreds of kilometers and zooming in a lot).
 
 Need to look into more advanced techniques like "relative to center" or "relative to eye"
 to improve the precision.
+
+.. seealso:: :py:func:`origin`
 %End
+
     QgsVector3D origin() const;
 %Docstring
-Returns coordinates in map CRS at which 3D scene has origin (0,0,0)
+Returns coordinates in map CRS at which 3D scene has origin (0,0,0).
+
+.. seealso:: :py:func:`setOrigin`
 %End
 
     QgsVector3D mapToWorldCoordinates( const QgsVector3D &mapCoords ) const;
@@ -91,11 +102,16 @@ Converts 3D world coordinates to map coordinates (applies offset and turns (x,y,
 
     void setCrs( const QgsCoordinateReferenceSystem &crs );
 %Docstring
-Sets coordinate reference system used in the 3D scene
+Sets coordinate reference system used in the 3D scene.
+
+.. seealso:: :py:func:`crs`
 %End
+
     QgsCoordinateReferenceSystem crs() const;
 %Docstring
-Returns coordinate reference system used in the 3D scene
+Returns coordinate reference system used in the 3D scene.
+
+.. seealso:: :py:func:`setCrs`
 %End
 
     QgsCoordinateTransformContext transformContext() const;
@@ -528,7 +544,6 @@ Sets DPI used for conversion between real world units (e.g. mm) and pixels
 .. versionadded:: 3.10
 %End
 
-
     double outputDpi() const;
 %Docstring
 Returns DPI used for conversion between real world units (e.g. mm) and pixels

python/3d/auto_generated/qgspointcloudlayer3drenderer.sip.in

@@ -10,7 +10,6 @@
 
 
 
-
 class QgsPointCloudLayer3DRenderer : QgsAbstractPointCloud3DRenderer
 {
 %Docstring(signature="appended")

python/3d/auto_generated/qgsvectorlayer3drenderer.sip.in

@@ -11,8 +11,6 @@
 
 
 
-
-
 class QgsVectorLayer3DRendererMetadata : Qgs3DRendererAbstractMetadata
 {
 %Docstring(signature="appended")

python/PyQt6/3d/auto_generated/qgs3dmapsettings.sip.in

@@ -16,6 +16,12 @@ class Qgs3DMapSettings : QObject, QgsTemporalRangeObject
 {
 %Docstring(signature="appended")
 Definition of the world.
+
+.. warning::
+
+   Qgs3DMapSettings are a QObject subclass, and accordingly are not
+   safe for access across different threads. See Qgs3DRenderContext instead
+   for a safe snapshot of settings from Qgs3DMapSettings.
 %End
 
 %TypeHeaderCode
@@ -74,10 +80,15 @@ with scenes spanning hundreds of kilometers and zooming in a lot).
 
 Need to look into more advanced techniques like "relative to center" or "relative to eye"
 to improve the precision.
+
+.. seealso:: :py:func:`origin`
 %End
+
     QgsVector3D origin() const;
 %Docstring
-Returns coordinates in map CRS at which 3D scene has origin (0,0,0)
+Returns coordinates in map CRS at which 3D scene has origin (0,0,0).
+
+.. seealso:: :py:func:`setOrigin`
 %End
 
     QgsVector3D mapToWorldCoordinates( const QgsVector3D &mapCoords ) const;
@@ -91,11 +102,16 @@ Converts 3D world coordinates to map coordinates (applies offset and turns (x,y,
 
     void setCrs( const QgsCoordinateReferenceSystem &crs );
 %Docstring
-Sets coordinate reference system used in the 3D scene
+Sets coordinate reference system used in the 3D scene.
+
+.. seealso:: :py:func:`crs`
 %End
+
     QgsCoordinateReferenceSystem crs() const;
 %Docstring
-Returns coordinate reference system used in the 3D scene
+Returns coordinate reference system used in the 3D scene.
+
+.. seealso:: :py:func:`setCrs`
 %End
 
     QgsCoordinateTransformContext transformContext() const;
@@ -528,7 +544,6 @@ Sets DPI used for conversion between real world units (e.g. mm) and pixels
 .. versionadded:: 3.10
 %End
 
-
     double outputDpi() const;
 %Docstring
 Returns DPI used for conversion between real world units (e.g. mm) and pixels

python/PyQt6/3d/auto_generated/qgspointcloudlayer3drenderer.sip.in

@@ -10,7 +10,6 @@
 
 
 
-
 class QgsPointCloudLayer3DRenderer : QgsAbstractPointCloud3DRenderer
 {
 %Docstring(signature="appended")

python/PyQt6/3d/auto_generated/qgsvectorlayer3drenderer.sip.in

@@ -11,8 +11,6 @@
 
 
 
-
-
 class QgsVectorLayer3DRendererMetadata : Qgs3DRendererAbstractMetadata
 {
 %Docstring(signature="appended")

src/3d/CMakeLists.txt

@@ -15,6 +15,7 @@ set(QGIS_3D_SRCS
   qgs3dmapsettings.cpp
   qgs3dmaptool.cpp
   qgs3dmapcanvas.cpp
+  qgs3drendercontext.cpp
   qgs3dsceneexporter.cpp
   qgs3dutils.cpp
   qgs3dwiredmesh_p.cpp
@@ -121,6 +122,7 @@ set(QGIS_3D_HDRS
   qgs3dmapscene.h
   qgs3dmapsettings.h
   qgs3dmaptool.h
+  qgs3drendercontext.h
   qgs3dsceneexporter.h
   qgs3dtypes.h
   qgs3dutils.h

src/3d/chunks/qgschunkedentity_p.cpp

@@ -59,8 +59,8 @@ static bool hasAnyActiveChildren( QgsChunkNode *node, QList<QgsChunkNode *> &act
 }
 
 
-QgsChunkedEntity::QgsChunkedEntity( float tau, QgsChunkLoaderFactory *loaderFactory, bool ownsFactory, int primitiveBudget, Qt3DCore::QNode *parent )
-  : Qgs3DMapSceneEntity( parent )
+QgsChunkedEntity::QgsChunkedEntity( Qgs3DMapSettings *mapSettings, float tau, QgsChunkLoaderFactory *loaderFactory, bool ownsFactory, int primitiveBudget, Qt3DCore::QNode *parent )
+  : Qgs3DMapSceneEntity( mapSettings, parent )
   , mTau( tau )
   , mChunkLoaderFactory( loaderFactory )
   , mOwnsFactory( ownsFactory )

src/3d/chunks/qgschunkedentity_p.h

@@ -65,7 +65,7 @@ class QgsChunkedEntity : public Qgs3DMapSceneEntity
     Q_OBJECT
   public:
     //! Constructs a chunked entity
-    QgsChunkedEntity( float tau, QgsChunkLoaderFactory *loaderFactory, bool ownsFactory,
+    QgsChunkedEntity( Qgs3DMapSettings *mapSettings, float tau, QgsChunkLoaderFactory *loaderFactory, bool ownsFactory,
                       int primitivesBudget = std::numeric_limits<int>::max(),
                       Qt3DCore::QNode *parent = nullptr );
     ~QgsChunkedEntity() override;

src/3d/mesh/qgsmesh3dentity_p.cpp

@@ -20,25 +20,25 @@
 #include <Qt3DRender/QGeometryRenderer>
 
 #include "qgsmeshlayer.h"
-#include "qgs3dmapsettings.h"
+#include "qgs3drendercontext.h"
 #include "qgsmesh3dmaterial_p.h"
 
 
 
-QgsMesh3DEntity::QgsMesh3DEntity( const Qgs3DMapSettings &map,
+QgsMesh3DEntity::QgsMesh3DEntity( const Qgs3DRenderContext &context,
                                   const QgsTriangularMesh &triangularMesh,
                                   const QgsMesh3DSymbol *symbol )
-  : mMapSettings( map )
+  : mRenderContext( context )
   , mTriangularMesh( triangularMesh )
   , mSymbol( symbol->clone() )
 {}
 
 QgsMeshDataset3DEntity::QgsMeshDataset3DEntity(
-  const Qgs3DMapSettings &map,
+  const Qgs3DRenderContext &context,
   const QgsTriangularMesh &triangularMesh,
   QgsMeshLayer *meshLayer,
   const QgsMesh3DSymbol *symbol )
-  : QgsMesh3DEntity( map, triangularMesh, symbol ),
+  : QgsMesh3DEntity( context, triangularMesh, symbol ),
     mLayerRef( meshLayer )
 {}
 
@@ -54,7 +54,7 @@ void QgsMeshDataset3DEntity::buildGeometry()
     return;
 
   Qt3DRender::QGeometryRenderer *mesh = new Qt3DRender::QGeometryRenderer;
-  mesh->setGeometry( new QgsMeshDataset3DGeometry( mTriangularMesh, layer(), mMapSettings.temporalRange(), mMapSettings.origin(), mMapSettings.extent(), mSymbol.get(), mesh ) );
+  mesh->setGeometry( new QgsMeshDataset3DGeometry( mTriangularMesh, layer(), mRenderContext.temporalRange(), mRenderContext.origin(), mRenderContext.extent(), mSymbol.get(), mesh ) );
   addComponent( mesh );
 }
 
@@ -67,7 +67,7 @@ void QgsMeshDataset3DEntity::applyMaterial()
     if ( datasetGroupIndex >= 0 )
       mSymbol->setColorRampShader( rendererSettings.scalarSettings( datasetGroupIndex ).colorRampShader() );
   }
-  QgsMesh3DMaterial *material = new QgsMesh3DMaterial( layer(), mMapSettings.temporalRange(), mMapSettings.origin(), mSymbol.get(), QgsMesh3DMaterial::ScalarDataSet );
+  QgsMesh3DMaterial *material = new QgsMesh3DMaterial( layer(), mRenderContext.temporalRange(), mRenderContext.origin(), mSymbol.get(), QgsMesh3DMaterial::ScalarDataSet );
   addComponent( material );
 }
 
@@ -76,28 +76,27 @@ QgsMeshLayer *QgsMeshDataset3DEntity::layer() const
   return qobject_cast<QgsMeshLayer *>( mLayerRef.layer.data() );
 }
 
-QgsMesh3DTerrainTileEntity::QgsMesh3DTerrainTileEntity(
-  const Qgs3DMapSettings &map,
-  const QgsTriangularMesh &triangularMesh,
-  const QgsMesh3DSymbol *symbol,
-  QgsChunkNodeId nodeId,
-  Qt3DCore::QNode *parent )
+QgsMesh3DTerrainTileEntity::QgsMesh3DTerrainTileEntity( const Qgs3DRenderContext &context,
+    const QgsTriangularMesh &triangularMesh,
+    const QgsMesh3DSymbol *symbol,
+    QgsChunkNodeId nodeId,
+    Qt3DCore::QNode *parent )
   : QgsTerrainTileEntity( nodeId, parent )
-  , QgsMesh3DEntity( map, triangularMesh, symbol )
+  , QgsMesh3DEntity( context, triangularMesh, symbol )
 {}
 
 void QgsMesh3DTerrainTileEntity::buildGeometry()
 {
   Qt3DRender::QGeometryRenderer *mesh = new Qt3DRender::QGeometryRenderer;
-  mesh->setGeometry( new QgsMeshTerrain3DGeometry( mTriangularMesh, mMapSettings.origin(), mMapSettings.extent(), mSymbol.get()->verticalScale(), mesh ) );
+  mesh->setGeometry( new QgsMeshTerrain3DGeometry( mTriangularMesh, mRenderContext.origin(), mRenderContext.extent(), mSymbol.get()->verticalScale(), mesh ) );
   addComponent( mesh );
 }
 
 void QgsMesh3DTerrainTileEntity::applyMaterial()
 {
   QgsMesh3DMaterial *material = new QgsMesh3DMaterial(
     nullptr, QgsDateTimeRange(),
-    mMapSettings.origin(),
+    mRenderContext.origin(),
     mSymbol.get(),
     QgsMesh3DMaterial::ZValue );
   addComponent( material );

src/3d/mesh/qgsmesh3dentity_p.h

@@ -21,7 +21,7 @@
 #include <Qt3DCore/QEntity>
 
 #include "mesh/qgsmesh3dgeometry_p.h"
-#include "qgs3dmapsettings.h"
+#include "qgs3drendercontext.h"
 #include "qgsmesh3dsymbol.h"
 #include "qgsterraintileentity_p.h"
 
@@ -52,13 +52,13 @@ class QgsMesh3DEntity
     void build();
   protected:
     //! Constructor
-    QgsMesh3DEntity( const Qgs3DMapSettings &map,
+    QgsMesh3DEntity( const Qgs3DRenderContext &context,
                      const QgsTriangularMesh &triangularMesh,
                      const QgsMesh3DSymbol *symbol );
 
     virtual ~QgsMesh3DEntity() = default;
 
-    Qgs3DMapSettings mMapSettings;
+    Qgs3DRenderContext mRenderContext;
     QgsTriangularMesh mTriangularMesh;
     std::unique_ptr< QgsMesh3DSymbol > mSymbol;
 
@@ -74,7 +74,7 @@ class QgsMeshDataset3DEntity: public Qt3DCore::QEntity, public QgsMesh3DEntity
 
   public:
     //! Constructor
-    QgsMeshDataset3DEntity( const Qgs3DMapSettings &map,
+    QgsMeshDataset3DEntity( const Qgs3DRenderContext &context,
                             const QgsTriangularMesh &triangularMesh,
                             QgsMeshLayer *meshLayer,
                             const QgsMesh3DSymbol *symbol );
@@ -94,7 +94,7 @@ class QgsMesh3DTerrainTileEntity: public QgsTerrainTileEntity, public QgsMesh3DE
     Q_OBJECT
 
   public:
-    QgsMesh3DTerrainTileEntity( const Qgs3DMapSettings &map,
+    QgsMesh3DTerrainTileEntity( const Qgs3DRenderContext &context,
                                 const QgsTriangularMesh &triangularMesh,
                                 const QgsMesh3DSymbol *symbol,
                                 QgsChunkNodeId nodeId,

src/3d/mesh/qgsmeshterraingenerator.cpp

@@ -23,9 +23,9 @@
 #include "qgsmeshlayer.h"
 #include "qgsmeshlayer3drenderer.h"
 #include "qgsterrainentity_p.h"
-#include "qgsterraintextureimage_p.h"
 #include "qgsmeshlayerutils.h"
-
+#include "qgs3dmapsettings.h"
+#include "qgs3drendercontext.h"
 
 QgsMeshTerrainTileLoader::QgsMeshTerrainTileLoader( QgsTerrainEntity *terrain, QgsChunkNode *node, const QgsTriangularMesh &triangularMesh, const QgsMesh3DSymbol *symbol )
   : QgsTerrainTileLoader( terrain, node )
@@ -37,7 +37,7 @@ QgsMeshTerrainTileLoader::QgsMeshTerrainTileLoader( QgsTerrainEntity *terrain, Q
 
 Qt3DCore::QEntity *QgsMeshTerrainTileLoader::createEntity( Qt3DCore::QEntity *parent )
 {
-  QgsMesh3DTerrainTileEntity *entity = new QgsMesh3DTerrainTileEntity( terrain()->map3D(), mTriangularMesh, mSymbol.get(), mNode->tileId(), parent );
+  QgsMesh3DTerrainTileEntity *entity = new QgsMesh3DTerrainTileEntity( Qgs3DRenderContext::fromMapSettings( terrain()->mapSettings() ), mTriangularMesh, mSymbol.get(), mNode->tileId(), parent );
   entity->build();
   createTexture( entity );
 
@@ -145,7 +145,7 @@ void QgsMeshTerrainGenerator::readXml( const QDomElement &elem )
   mSymbol->readXml( elem.firstChildElement( "symbol" ), rwc );
 }
 
-float QgsMeshTerrainGenerator::heightAt( double x, double y, const Qgs3DMapSettings & ) const
+float QgsMeshTerrainGenerator::heightAt( double x, double y, const Qgs3DRenderContext & ) const
 {
   return QgsMeshLayerUtils::interpolateZForPoint( mTriangularMesh, x, y );
 }

src/3d/mesh/qgsmeshterraingenerator.h

@@ -85,7 +85,7 @@ class _3D_EXPORT QgsMeshTerrainGenerator: public QgsTerrainGenerator
     QgsRectangle rootChunkExtent() const override;
     void writeXml( QDomElement &elem ) const override;
     void readXml( const QDomElement &elem ) override;
-    float heightAt( double x, double y, const Qgs3DMapSettings & ) const override;
+    float heightAt( double x, double y, const Qgs3DRenderContext &context ) const override;
 
   private slots:
     void updateTriangularMesh();

src/3d/qgs3dmapscene.cpp

@@ -469,7 +469,7 @@ void Qgs3DMapScene::createTerrainDeferred()
     const QgsAABB clippingBbox = Qgs3DUtils::mapToWorldExtent( mMap.extent(), rootBbox.zMin, rootBbox.zMax, mMap.origin() );
     mMap.terrainGenerator()->setupQuadtree( rootBbox, rootError, maxZoomLevel, clippingBbox );
 
-    mTerrain = new QgsTerrainEntity( mMap );
+    mTerrain = new QgsTerrainEntity( &mMap );
     mTerrain->setParent( this );
     mTerrain->setShowBoundingBoxes( mMap.showTerrainBoundingBoxes() );
 
@@ -645,7 +645,7 @@ void Qgs3DMapScene::addLayerEntity( QgsMapLayer *layer )
       tiledSceneRenderer->setLayer( static_cast<QgsTiledSceneLayer *>( layer ) );
     }
 
-    Qt3DCore::QEntity *newEntity = renderer->createEntity( mMap );
+    Qt3DCore::QEntity *newEntity = renderer->createEntity( &mMap );
     if ( newEntity )
     {
       newEntity->setParent( this );