-
Notifications
You must be signed in to change notification settings - Fork 4
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
Demo Analyze Everything Hung Up on Reconstruction #349
Demo Analyze Everything Hung Up on Reconstruction #349
Comments
When Cell 12 stops and Cell 13 can be clicked following error is displayed: IndexError Traceback (most recent call last) IndexError: list index out of range |
Hi @carlos-gomez00 , this PR should fix the second one of your problems. As for the first one, the PR fixes the link to the dashboard that's created so that it also works for the viewer. The root cause, in this case, is probably that for some reasons (will have to look into the code), the reconstruction is sampling waaaay too dense on the surface of the droplet and then has to trace-refine a myriad of points, which is unnecessary. I can change the code to ensure that the density of points on the surface is calculated in physical units which should alleviate the problem a bit (but that's a separate PR). |
@carlos-gomez00 I just looked into the data you were trying to analyze and found another issue: The If you do it from the viewer, this should be fine by default - checking why it was slow now. Edit: PR #351 should fix the astronomical computation times. If you don't want to wait until the next release, a quick fix would be to change the This parameter essentially determines how many points on the surface of the droplet are used for the tracing. I assume that the legacy STRES code always resamples the image to a voxel size of |
Hey @jo-mueller I left the workstation running and it did finish running the code, but it had strange results. Probably due to the axes order, I'll try changing it and running it again. Just to show what it did, I pasted the results below. But the rescaled image did not overlap with the previous image I'll upload with the results after fixing axis on the data |
This, at least, is something I'd expect. The input image doesn't show the correct physical pixel sizes, but the output does. You can fix it by passing the voxel sizes of the input data to the viewer in cell [9]: viewer.add_image(example_data, scale=[voxel_size_z, voxel_size_y, voxel_size_x]) |
Hi @carlos-gomez00 , can I close this issue? |
@carlos-gomez00 |
Hi @jo-mueller I have tried to run the Notebook again and this is the error I obtained, the reconstruction didn't show this time I didn't understand the error as it was set to cubic |
Issue in: docs/01_code_usage/01_toolboxes/demo_analyze_everything.ipynb
Protocol: (1) Ran Juypter-lab from Miniforge prompt; (2) Opened demo_analyze_everything.ipynb; (3) Edited Cell 3, 4, and 7; (4) Ran code
=> When reaching Cell 12, the code is hung up and causes napari to stop responding.
I have attached the messages from the terminal window. Not sure if this is a computer admin problem or an issue of the parameters I input into the code.
The text was updated successfully, but these errors were encountered: