[jcelerier]

These are the engine fixes found while building the shading tiers (unlit / blinn_phong / pbr_fast / classic_pbr) in csf-examples, which use the Scene Preprocessor's aux buffers. Each fix comes with a test that fails without it and passes with it.

Raw raster and CSF

• AUXILIARY and OUTPUTS size expressions that read an input are evaluated; VERTEX_COUNT and INSTANCE_COUNT take expressions.
• Top-level PIPELINE_STATE.COLOR_WRITE is honoured. The front face stays correct under CULL_MODE none.
• An AUXILIARY texture without MIPMAP_MODE reads its mips. A STATIC image input fed by Images or a video reads the image.
• A CSF processes every mesh of a multi-mesh geometry. It also reads packed and interleaved attributes at their own stride. An attribute it cannot repack reads as zeros.
• A vertex shader with no inputs keeps no vertex bindings. MULTIVIEW shaders compile and render on OpenGL.
• A geometry input with several cables warns.

Scene

• The Scene Preprocessor draws meshes whose index buffer is on the GPU. It copies GPU color0 and texcoord1, and warns once when it skips 16-bit GPU indices.
• Scenes on one port merge in edge order.
• Shadow cascades belong to the light that Shadow Cascade Setup fits them to: light_slot, see the libossia PR.
• Without an Environment node there is no ambient light.
• glTF and FBX lights light the scene, and cast shadows when the file says so.
• Alpha-blended draws come after the opaque ones, back to front from the centre of the scene's cameras, with no depth write.
• glTF normal scale, occlusion strength and clearcoat normal scale reach the shaders. glTF skins keep their joint hierarchy and rest pose.

Nodes

• PBR Mesh and Material Override take their base and emissive colours from colour controls. PBR Mesh keeps its geometry's auxiliaries and transform, and Material Override takes effect on it.
• Environment sets the fog mode and density.
• Shadow Cascade Setup takes the light that casts, and covers the scene without a camera.
• Configure Primitive's ** glob spans path segments.
• A CPU texture outlet publishes its texture, so Texture Info reads the real size. Texture to buffer copies a CPU producer's texels.
• A script can set a texture inlet's format (inl.textureFormat = "RGBA16F").
• Half-float planar EXRs decode.

Added after review of the first CI run

• CSF storage declared PERSISTENT reads the previous frame as <name>_prev.
• Model Display draws a mesh white when nothing is wired to its texture inlet.
• On Metal, the Scene Preprocessor widens 16-bit GPU indices with a compute pass instead of skipping the mesh.
• Builds on every supported Qt: the blend pass no longer uses setDepthClamp or QRhiResource::rhi(), and GPU JS needs Qt 6.7, which has QQuickRhiItem. Checked locally with mold:
  • Qt 6.4.2 with the test suite and dynamic plugins;
  • Qt 6.5.3 with PCH;
  • Qt 6.9.1 as a unity build;
  • the SDK's Qt 6.12.

Depends on

• geometry: shadow cascades name their light; no ambient without an environment; clearcoat normal scale libossia#958, give some space for the states' little plus #959 and minimap issues #960 (all merged): the submodule is pinned to libossia master d606b8801.
• examples: custom-pbr uses PBR Mesh's colour controls score-docs#95: it migrates custom-pbr.score to the PBR Mesh colour controls.
• Outlet nodes named by their UID; keep liblsl's symbols apart from score's static link score-addon-lsl#9 (bumps liblsl to the merged cmake: keep a static liblsl's internals out of the linking program jcelerier/liblsl#1): test_app_hides_lsl_symbols expects it.
• ossia/sdk: qtbase patches 0008–0011, qtshadertools 0001–0002 and qtquick3d 0002 (all on master). Linux GL multiview needs a CI build that includes qtshadertools 0002.
• score-addon-academy#4 goes in after this PR: it uses Gfx/Graph/RowOrder.hpp.

Validation

• Linux: Vulkan and OpenGL, build-sdk43. A full ctest run at 7f2252111d, then re-runs of every failure. Still failing: GL multiview (needs the SDK Qt above), two explorer tests that depend on the desktop window manager (they pass on Xvfb), and the PipeWire and shader-sweep tests that already failed before this PR.
• macOS (Metal) and Windows (D3D11/12, Vulkan, GL): the example-score sheet was rendered at 132c8b3330. Rebuilds at the current head are in progress.
• The csf-examples tier, gallery and depth scenes were re-rendered against this branch.

Tests on macOS, Windows and static builds (folded in from #2332):

• ForkProbe.hpp: resolve __llvm_profile_write_file with dlsym on Apple, as ObjectGallery.cpp does. Mach-O has no weak undefined externals, so seven tests failed to link on macOS outside coverage builds.
• bitfocus_module_driver is a GUI test: it boots score_test/App.hpp, which with static plug-ins references every plug-in.
• CanvasPaintingTest: near renamed to colorsClose (<windef.h> defines near).
• Regex_UI_Test: its Switch uuid no longer clashes with Switch_UI_Test's in unity builds.
• The test fixture deletes parentless dialogs between applications: the local device's "port taken" box, raised for the port 0 the tests ask for, outlived its QApplication and crashed the next one under xcb (8 integration tests).
• macOS integration tests use DYLD_FALLBACK_LIBRARY_PATH, since score refuses to start with DYLD_LIBRARY_PATH set (18 tests).
• score::interactiveSession() moves to score/widgets/MessageBox and the local device uses it: with --script, offscreen or minimal platforms, the port warning is only logged.

Verified together with #2330 on Linux (Vulkan, GL), macOS (Metal) and Windows (D3D11/12, Vulkan, GL) at the same commits.

D3D12 robustness:

• The Scene Preprocessor merges the mesh-slab uploads of a scene load (one per contiguous run of each stream instead of one per primitive and stream): Sponza went from about 770 uploads into the 128 MB mesh buffers to a handful. Each upload is a staging allocation and a copy on every backend; Qt's D3D12 backend staged the whole buffer per upload, which ran the process out of memory (48 GB, then a lost device). Peak shared GPU memory is now 1.25 GB even on the current SDK's Qt. Vulkan images are unchanged, pixel for pixel.
• Outputs notice a D3D11/D3D12 device removed under offscreen frames (checkDeviceLost asks the native device for GetDeviceRemovedReason). Qt only reports a loss when a Present or a texture creation fails, so offscreen outputs kept rendering into a dead device until killed.
• The two Qt bugs behind this (staging sized by the buffer; sRGB mipmaps through typed UAVs) are patched in ossia/sdk 7da23a0 (qt-patches 0006 and 0007). With both, the Sponza and DamagedHelmet scenes that lost the device render on D3D12 within 0.2% of Vulkan.

Backend portability, from the contact-sheet triage of the example scores:

• Row order: every published 2D image has the row order of a rendered texture. CPU uploads are flipped where the RHI has Y up in NDC (OpenGL, Metal, D3D), in Gfx/Graph/RowOrder.hpp.
• Alpha: render targets are premultiplied, file textures straight. Texture-reference sampling un-premultiplies render-target sources. Base-colour textures of MASK/BLEND materials get their transparent texels' colour bled in before their mip chain is built.
• Gaussian splats draw on every backend and keep render-target row order on Metal and D3D.
• Identifiers the target language reserves are renamed instead of refusing the shader: HLSL and MSL words, plus the GLSL words NVIDIA's GL compiler rejects (char, register, row_major, …).
• Direct3D:
  • point sprites, drawn as instanced quads;
  • per-view PER_MIP passes with lowered multiview;
  • readonly storage blocks as SRVs on D3D11;
  • no resource bound as both halves of an in-place pair;
  • DRED breadcrumbs and page faults are logged when a D3D12 device is lost.
• Points below a pixel draw on Metal. ISF depth macros read mip 0, and IMG_NORM_PIXEL_LOD reads an explicit level.
• With the SDK's Qt patches, all 58 problem scores from the triage render upright on Linux Vulkan/GL, Metal, D3D11, D3D12 and Windows Vulkan/GL.

Merged in: #2331 (camera release, Syphon decoder on Metal, an inert replaced execution context) and #2330 (static clang64 test build, add-ons through response files on Windows).

Crash and build fixes from the full Linux test run:

• Model Display no longer draws with a vertex buffer that its producer released during a render-list rebuild. This use-after-free crashed model_display_gpu_geometry in 9 of 15 runs, and the same crash was seen on macOS.
• Boost.Asio's global symbols are versioned on every platform (BOOST_ASIO_ENABLE_VERSION_NAMESPACE). liblsl's standalone Asio had replaced Boost's thread entry point, so mapper_http and mapper_datagram double-freed.
• A render list's first frame no longer submits init-time uploads into a buffer it just freed. When an inlet's spec changed before that frame, the renderer was re-initialised and its material UBO deleted while the initial update batch still pointed at it. input_sampler_filter (raw raster, OpenGL) crashed or read back black in about half of its runs. The batch is now committed as soon as the frame takes it, with a deterministic regression case in outside_frame_release_buffer.
• Path resolution in GPU node controls applies only to file and folder choosers. A string control is no longer turned into a path because it starts with a root token.
• score's fallback VkDevice, used when the GPU has no Vulkan Video decode queue or with SCORE_GFX_NO_SHARED_VKDEVICE, enables DMA-BUF and DRM-modifier import again. vulkan_importer_device_checks imports a LINEAR udmabuf and a vendor-tiled GBM buffer there with no validation errors.
• Syphon input no longer crashes when the sender quits (Metal, seen in Domeport Pro): the borrowed texture was freed while the sampler still bound it. The last frame stays on screen, and a restarted sender reconnects under its new UUID.
• Linux hides all of the static LLVM, clang and Polly. Mesa's libLLVM.so.20 was binding to score's LLVM 23 and crashed at exit or in llvmpipe shader compiles.
• QT3D_RENDERER and the Quick Controls style are set before plug-ins start threads. Adding a variable reallocates the environment on glibc < 2.41, and LeapC's poll thread was reading it at the time.
• avendish: enum ports start at their declared default.
• Tests:
  • the uniform_input fixture gets its QRhi;
  • the canvas painter's parts load through its qmldir;
  • ObjectGallery links the Qt platform plug-ins;
  • the scenario presenter is looked up again after adding a root slot;
  • media clips cover the pixel formats that still go through swscale;
  • text_render goldens are regenerated;
  • new test_app_hides_lsl_symbols.

Spout (Windows):

• D3D11, D3D12 and Vulkan outputs register through a full-size name buffer. When the name was taken, Spout renamed it in place to name_1 inside a temporary std::string, two bytes past the heap block for a 31-character name. Score then released the configured name, which removed the other application's sender from the registry. Score now keeps the registered name, uses it for release and GL frame sync, and logs the rename.
• The GL input checks IsUpdated() after each receive, as the SDK requires, and follows sender size and format changes from the receiver.

Vulkan interop on Qt Quick's own VkDevice:

• Every Qt Quick window asks its VkDevice for VK_KHR_external_memory (plus _win32 on Windows), VK_KHR_dedicated_allocation and VK_KHR_get_memory_requirements2. An application event filter merges them into the window's graphics configuration on SurfaceCreated, before the first expose creates the QRhi, and keeps whatever the app already configured. If a window already has a QRhi at that point, the filter logs a warning. A Spout input shown by a QML TextureSource on Vulkan rendered black (vkGetMemoryWin32HandlePropertiesKHR not available) and now shows the sender's pixels.
• The DMA-BUF, host-import, CUDA-decoder and CUDA-semaphore importers decide availability through the QRhi's own device (vulkanDeviceCommand(), i.e. vkGetDeviceProcAddr) instead of the instance. vkGetInstanceProcAddr hands out a loader trampoline for any device command. On a Qt-default device, all four reported available, and the host import and semaphore export jumped to a null dispatch entry (SIGSEGV). They now report unavailable, and the existing fallbacks run. On score's own device, all four are still available.
• SpoutInput uses the QRhi's own QVulkanInstance: a Qt Quick device belongs to QVulkanDefaultInstance.
• New test_gfx_vulkan_importer_device_checks.

Packaging:

• tools/create-app.sh resolves relative --app-environment, icon, qrc and appdata paths against the caller's directory, and stops at parse time when the environment file is missing. A relative environment path used to work only when a per-OS overlay existed.
• Windows custom apps get an explicit AppUserModelID: --app-identifier, or ossia.score.custom.<name> without it. The launcher passes it to app-bin.exe, which sets it on the process before any window exists. Every top-level window also carries it, with relaunch properties that point at the launcher. The running app and the launcher now share one taskbar identity, and pinning the running app pins the launcher. Plain score is unchanged.

Release review (fixups, to autosquash with git rebase -i --autosquash origin/master; each comes with a test that fails without it)

• cbd2ce7 → beefbb6: PBR Mesh no longer forwards a Scene Preprocessor's _blend_draw_cmds as an injection (test_threedim_pbr_mesh_material).
• 1c20da2 → f8ba9df: row_major inside layout(...) is a qualifier and stays; glslang rejected the renamed form on OpenGL (test_gfx_shader_corpus_targets).
• fdcbbf1 → 54cf1ef: SCORE_AUDIO_NO_INPUT applies: setupDefaultSettings writes the CardIn default first, so the unconfigured state is the empty card, not a missing key (test_unit_audio_settings_no_input).
• 8446374 → 97430d4, 04b1c6a → f8ba9df, a6ec6be → 02f03a2: ossia::hash_set / ossia::hash_map where the branch introduced std containers.

Review notes, unconfirmed doubts and design items (per-frame per-index copies for 16-bit GPU indices, the blend sort versus the MDI fast path): /mnt/build/diag/release-review-2326.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JeL19ofVgPe9u9q7J1V9ex

[coveralls]

Coverage is 48.861% — gfx/shading-tiers into master. No base build found for master.