Skip to content

Conversation

@PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Nov 7, 2025

Implements ?config="myconfig.json" URL parameter to ensure configuration files are loaded before data files, eliminating race conditions when both are in a ?files paramter

Fixes #817

@netlify
Copy link

netlify bot commented Nov 7, 2025

Deploy Preview for volview-dev ready!

Name Link
🔨 Latest commit fa73f8f
🔍 Latest deploy log https://app.netlify.com/projects/volview-dev/deploys/6913523266ed6400095cb3ba
😎 Deploy Preview https://deploy-preview-818--volview-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PaulHax PaulHax requested a review from zachmullen November 7, 2025 19:38
@PaulHax PaulHax changed the title feat: add config URL parameter to load configs before data Add config URL parameter and automatic layering by file name Nov 11, 2025
Implements ?config= URL parameter to ensure configuration files
are loaded before data files, eliminating race conditions when
both are specified.

Fixes #817
Implements automatic layering based on file name patterns.
When loading files with matching names like base-image.nii and base-image.layer.nii, the layer file is automatically placed on top of the base image.
Sort matching layers and segment groups alphabetically by filename to provide deterministic ordering. Users can control stacking order using numeric prefixes in filenames (e.g., file.layer.1.nii, file.layer.2.nii).

Updated documentation to clarify naming patterns and show that file formats don't need to match between base image and associated files.
@PaulHax PaulHax added this pull request to the merge queue Nov 11, 2025
Merged via the queue into main with commit 1a0c286 Nov 11, 2025
7 checks passed
@zachmullen zachmullen deleted the config-race branch November 11, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Config Loading Race Condition

2 participants