Forrest Li

Results 55 comments of Forrest Li

vtkPaintWidget changed API slightly. You should no longer need the method `updatePaintbrush`, as vtkPaintWidget now handles the paintbrush orientation internally.

vtk.js v15 updates to webpack 5, and with it `worker-loader` options object changed. Assuming react-vtkjs-viewport is using the vtk dependency file, that shouldn't be happening. Can you print out the...

I've run into this issue as well. As a workaround, I've patched the `funcToSource` method to wrap the function as a function invocation, rather than extracting the function code. The...

I think providing a memory-bounded version would be difficult, especially in the context of generating a volume from DICOM slices. As for how much RAM readImageDicomFileSeries takes, that will need...

If passing in an already sorted series will reduce peak memory usage, one idea is to have a utility that can take in a list of files and return them...

Some notes and questions on further investigation: - What memory are you measuring/how are you measuring your allocated memory? Are you measuring the main thread, the web worker, or the...

> if the browser stops responding (white screen) it is usually the main thread OOM This is an interesting case. The webworker should just throw an error, but not crash,...

I will also make a follow-up suggestion: either a repo containing a minimal reproduction scenario or any console errors (or both) would be very helpful for us in debugging this...

Yes! The necessary change is in #606.

For itkwidgets, browser tabs usually have a ceiling on the memory allocation, though that usually results in a browser tab crash and not a blank image. If it's a TIF...