Skip to content
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

[WIP] Oit weighted - generalization #2230

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

Conversation

brisvag
Copy link
Collaborator

@brisvag brisvag commented Oct 7, 2021

I'm starting this PR on the main repo in order to get feedback. This is a continuation on #2039, attempting to implement a generalization where each Visual provides an is_transparent flag.

The new code is not clean (a lot of ugly isinstance and hasattr), but should work and provides a proof of concept.

Unfortunately, I tried to add markers to the example and things fail; specifically, it's due to a couple of line (which @asnt already identified as problematic) where _prepare_draw needs to be called with no view (which causes all sort of error if the view is needed).

@brisvag brisvag marked this pull request as draft October 7, 2021 15:42
@brisvag
Copy link
Collaborator Author

brisvag commented Oct 8, 2021

Ok, I pushed some more changes with very, very ugly patches to make Markers work. Take this just as a way to show what goes wrong and where.

If you try the example now, you'll see that the markers show up, but are behaving strangely. Is this just a limitation of this kinf of rendering, due to the fact that points are actually opaque except for the aliasing edge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants