-
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
Shape selector #91
Shape selector #91
Conversation
When launching the plugin, I get this error:
I installed into an existing environment. |
Oops, fixed |
…attice into shape-selector
With cropping:
|
I realised why this happens. It's because the Shape has 3 dimensions but the ROI class I created only expects 2 dimensions. We don't support ROIs in 3D or higher do we? If not, I'll just drop the other dimensions. shape.get_array()
array([[ 27.83333333, 540.61463643, 420.71545234],
[ 27.83333333, 540.61463643, 1006.41766134],
[ 27.83333333, 992.6239499 , 1006.41766134],
[ 27.83333333, 992.6239499 , 420.71545234]]) |
Yes, the ROI will be in 2D for now (X,Y). We can crop in 3D by specifying z range. But I think z range is disabled for now. |
Thanks If I use the New Crop button under Crop tab, cropping works fine when I draw ROIs. But, when selecting ROIs, the corresponding cell doesn't get selected or updated though. If I create a Shapes layer manually and click mouse button to draw an ROI, I get an error with the following traceback
I can draw ROIs after this without any issues, but I have 2 errors: with traceback:
|
Hmm, I'm struggling to reproduce this. Can you maybe take a screenshot when it happens, and also list the minimal actions you have to take beforehand to trigger this issue? |
shape_selector.mp4 |
--
-- -Create SHapes layer via
The ROI selection doesn't work, but cropping works based on selection |
If I import an ImageJ/Fiji ROIManager file, it works fine. The selected ROI is highlighted and cropping works.. shape_selector_import_roimanager.mp4 |
Hmm, I still can't reproduce this even with the same steps. Does the image have any unique properties? |
I tried with different images actually. |
Btw, did you try it on a Windows PC? |
So, I'm still getting the error. Also, I didn't open any images either test.mp4napari info:
|
Okay, I think I've fixed this issue on Windows. I've tested it on the VM as well. |
I can't replicate this on the latest commit. Let me know if it's still there when you next test this. |
I can confirm its working now and the cropping works perfectly as well. |
Closes #90.