From ebbd000714942adcc0709381eaef0e330ce4919c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Candice=20Bent=C3=A9jac?= Date: Tue, 17 Dec 2024 19:07:40 +0100 Subject: [PATCH] [Viewer3D] MediaLoader: Send `pointSize` as is `pointSize` was divided by 100 before being sent to the shader that rendered the programmable points, and was used as is to set the size of the "fixed size" points implicitly in the QML. Now that both cases are handled by the same shader, we do not need to perform this division here: it will be done directly in the shader (by opposition, keeping it here would have meant performing a x100 multiplication in the shader for the "fixed size" case). --- meshroom/ui/qml/Viewer3D/MediaLoader.qml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meshroom/ui/qml/Viewer3D/MediaLoader.qml b/meshroom/ui/qml/Viewer3D/MediaLoader.qml index 98f95bad31..986eaf2031 100644 --- a/meshroom/ui/qml/Viewer3D/MediaLoader.qml +++ b/meshroom/ui/qml/Viewer3D/MediaLoader.qml @@ -109,7 +109,7 @@ import Utils 1.0 var obj = Viewer3DSettings.sfmDataLoaderComp.createObject(sfmDataLoaderEntity, { "source": source, "fixedPointSize": Qt.binding(function() { return Viewer3DSettings.fixedPointSize }), - "pointSize": Qt.binding(function() { return 0.01 * Viewer3DSettings.pointSize }), + "pointSize": Qt.binding(function() { return Viewer3DSettings.pointSize }), "locatorScale": Qt.binding(function() { return Viewer3DSettings.cameraScale }), "cameraPickingEnabled": Qt.binding(function() { return root.enabled && root.cameraPickingEnabled }), "resectionId": Qt.binding(function() { return Viewer3DSettings.resectionId }),