-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] Large series will not render as volumes and will crash (Array Buffer Allocation Failed) #4150
Comments
I can share the data on slack if needed |
@IbrahimCSAE - yes, if you shared the data I would look into this issue. Do I have time against flexview for it? I'm also doing some other work that is related. |
Hi all, is there a defined strategy on this ? In my opinion it will be always a series that will not load in memory, either because browser limitation, hardware limitation, or series anyway too large with thousands of instances |
Bill added a nice PR to fix the issue until 4GB for now cornerstonejs/cornerstone3D#1260 |
That's cool ! (but i think it will have a day when we will have > 4gb images ^^⁾ |
see you then :D |
@salimkanoun - I agree, we will have > 4GB images - we can double the new size on most machines now, by switching to 16 bit textures (instead of float 32). That seems to have largely been fixed on the GPU. Getting to larger than 4 gb will require either WebAssembly64, or webgpu implementation, which is in the works inside VTK, but has a LOT of testing and fixes to get things stable. I'm also looking at splitting volumes out per image, which splits the webgl versus browser memory, and that might resolve things too, but I'm not sure when we will have time for that one. |
Describe the Bug
If a series is a large enough the array buffer allocation fails and we can't do MPR on it.
Chrome:
Safari can do better but still the render seems odd, I also have to override the cs3d cache first to increase it:
Steps to Reproduce
The current behavior
The viewer crashes
The expected behavior
Render MPR correctly
OS
MacOS
Node version
18
Browser
Chrome
The text was updated successfully, but these errors were encountered: