-
Notifications
You must be signed in to change notification settings - Fork 8
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
reducing coordinate opacity makes black dots on the image layer #143
Comments
Confirm I can't reproduce on my machine. Can you paste the output of |
Voila:
|
For now we just need to find a minimal reproducer; I think it's an openGL issue, so it's unlikely to come out of napari code specifically, I'll probably have to dig into vispy. Try this: import napari
import numpy as np
v = napari.Viewer()
v.add_image(np.random.rand(100, 100))
v.add_points(np.array([50, 50]), opacity=0.4, shading='spherical') and if it's still fine, change the point colors to 'teal'? |
This also works fine, as well as changing colors to teal. |
Nasty bug :P some other things to try:
|
will try it tomorrow morning! |
I tried those things but don't get the issue. Also tried making it 3D and using the out_of_slice_display (as below), but also does not reproduce it.
|
Ok, I actually got it to reproduce. In my case, it's indeed because of the layer order. Basically, if you have the image above the points and make the points transparent, the thing you see behind the points is the canvas (black). This is actually kind of unsolvable because of how opengl blending works (unless we break other things). Here's a comment mentioning this exact issue (and approrpiately, it was me who said "let's cross that bridge when we get to it" xD). So for you the solution should be: move the points layer above the images. Does that work? |
It's a been while, sorry for that. That indeed solves the problem haha |
I already have a little trick like that in fact (sorting layers before adding them), the problem is, it doesn'y work xD napari gives the files to readers one by one instead of as a list... Not sure there's a solution to that other than writing my own entry point script. |
Here, a short video of the issue (recorded directly after I opened Blik):
Screencast.from.02-08-23.16.49.56.webm
The text was updated successfully, but these errors were encountered: