Skip to content

Incremental DICOM loading - #606

Merged
floryst merged 46 commits into
mainfrom
streaming-base
Apr 16, 2025
Merged

Incremental DICOM loading#606
floryst merged 46 commits into
mainfrom
streaming-base

Conversation

@floryst

@florystfloryst commented Jun 13, 2024

Copy link
Copy Markdown
Contributor

This PR updates VolView to support incremental DICOM loading. This is a pretty big change, so I hope to improve this issue description with details of how this is broken down.

Components:

@netlify

netlifyBot commented Jun 13, 2024

Copy link
Copy Markdown

Deploy Preview for volview-dev ready!

NameLink
🔨 Latest commit95aaa94
🔍 Latest deploy loghttps://app.netlify.com/sites/volview-dev/deploys/67ffc13befda1500089bd3fb
😎 Deploy Previewhttps://deploy-preview-606--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 site configuration.

Comment threadsrc/io/import/processors/updateUriType.ts
@floryst
florystforce-pushed the streaming-base branch 2 times, most recently from 617a675 to 0bfe9d2CompareAugust 1, 2024 19:06
@floryst
florystforce-pushed the streaming-base branch 2 times, most recently from f854153 to 3c79e69CompareMarch 19, 2025 17:42
@PaulHax

This comment was marked as resolved.

@floryst

Copy link
Copy Markdown
ContributorAuthor

There is an error loading a volview.zip with sample data.

  1. Load prostate
  2. Save volview.zip
  3. Refresh
  4. Load volview.zip

Error:

loggers.ts:13 Error: Could not find the parent of an archive source
at deserialize (restoreStateFile.ts:214:17)
at rebuildDataSources (restoreStateFile.ts:261:22)
at async restoreDatasets (restoreStateFile.ts:321:39)
at async restoreStateFile (restoreStateFile.ts:373:30)
at async evaluateChain (evaluateChain.ts:18:22)
at async importDataSources (importDataSources.ts:144:38)
at async load (loadUserFiles.ts:236:18)
at async wrapper (loadUserFiles.ts:289:9)

I just pushed 6e75e97, which addresses this issue.

@floryst
floryst marked this pull request as ready for review April 9, 2025 20:30
@floryst
floryst added this pull request to the merge queueApr 16, 2025
Merged via the queue into main with commit 7f2a992Apr 16, 2025
PaulHax pushed a commit to PaulHax/VolView that referenced this pull request Apr 24, 2025
@floryst
floryst deleted the streaming-base branch June 27, 2025 14:54
Sign up for freeto 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.

Image range and histogram computation: cache, defer and/or optimize

2 participants

@floryst@PaulHax