review

Author: Meakk

.github/actions/generic-ci/action.yml

@@ -198,7 +198,7 @@ runs:
         -DF3D_TESTING_ENABLE_EGL_TESTS=${{ runner.os == 'Linux' && 'ON' || 'OFF' }}
         -DF3D_TESTING_ENABLE_EXTERNAL_GLFW=${{ (runner.os == 'Linux' && inputs.rendering_backend == 'auto') && 'ON' || 'OFF' }}
         -DF3D_TESTING_ENABLE_EXTERNAL_QT=${{ (runner.os == 'Linux' && inputs.rendering_backend == 'auto') && 'ON' || 'OFF' }}
-        -DF3D_TESTING_ENABLE_ILLUSTRATION_TESTS=${{ (runner.os == 'Linux') && 'ON' || 'OFF' }}
+        -DF3D_TESTING_ENABLE_ILLUSTRATION_TESTS=${{ (runner.os == 'Linux' && inputs.vtk_version == 'commit') && 'ON' || 'OFF' }}
         -DF3D_TESTING_ENABLE_GLX_TESTS=${{ (runner.os == 'Linux' && inputs.rendering_backend == 'auto') && 'ON' || 'OFF' }}
         -DF3D_TESTING_ENABLE_LONG_TIMEOUT_TESTS=${{ (runner.os == 'Linux' || runner.os == 'Windows') && 'ON' || 'OFF' }}
         -DF3D_TESTING_ENABLE_OSMESA_TESTS=${{ runner.os == 'Linux' && 'ON' || 'OFF' }}

application/testing/tests.documentation.cmake

@@ -44,7 +44,23 @@ file(DOWNLOAD
     SHOW_PROGRESS
 )
 
-## Wrapper that sets many default arguments for documentation illustration tests
+#[==[.md
+# Wrapper that sets many common arguments for documentation illustration tests
+
+Usage:
+
+```
+f3d_doc_test(<NAME> [ARGS...])
+```
+  - `UI` Mark the test to require the presence of UI component and disable it otherwise
+  - `BACKGROUND` The background is visible, otherwise it's transparent
+  - `ROTATE` Rotate the camera to show the model from a different angle, otherwise it's front facing
+  - `NAME` Provide the name of the test, mandatory and must be unique
+  - `THRESHOLD` Provide the `--reference-threshold` to use instead of the default
+  - `DATA` Data to open, support multiple input, also set test labels accordingly
+  - `REF_IMAGE` Provide the reference image name, saved into `doc/user/images`
+  - `ARGS` Supplement arguments to add to the f3d command line
+#]==]
 function(f3d_test_doc)
   cmake_parse_arguments(F3D_TEST_DOC "UI;BACKGROUND;ROTATE" "NAME;DATA;REF_IMAGE;THRESHOLD" "ARGS" ${ARGN})
 
@@ -290,7 +306,7 @@ f3d_test_doc(NAME TestDocHDRICOC50 DATA DamagedHelmet.glb REF_IMAGE hdri_coc_50.
 
 ## --light-intensity
 f3d_test_doc(NAME TestDocLightIntensity1 DATA DamagedHelmet.glb REF_IMAGE light_intensity_1.png ROTATE)
-f3d_test_doc(NAME TestDocLightIntensity05 DATA DamagedHelmet.glb REF_IMAGE light_intensity_0.5.png ROTATE ARGS --light-intensity=0.5)
+f3d_test_doc(NAME TestDocLightIntensity2 DATA DamagedHelmet.glb REF_IMAGE light_intensity_2.png ROTATE ARGS --light-intensity=2)
 
 ## --scalar-coloring
 f3d_test_doc(NAME TestDocScalarColoringOFF DATA dragon.vtu REF_IMAGE scalar_coloring_off.png ROTATE ARGS)

doc/user/03-OPTIONS.md

@@ -715,9 +715,9 @@ _Adjust the intensity_ of every light in the scene (except HDRI).
 
 #### compare
 
-| 1 (default)                         | 0.5                                   |
-| ----------------------------------- | ------------------------------------- |
-| ![](./images/light_intensity_1.png) | ![](./images/light_intensity_0.5.png) |
+| 1 (default)                         | 2                                   |
+| ----------------------------------- | ----------------------------------- |
+| ![](./images/light_intensity_1.png) | ![](./images/light_intensity_2.png) |
 
 ## Scientific visualization options