Skip to content

Improve stackgl rendering pipeline for transparent traces - #4643

Open
archmoj wants to merge 4 commits into
mainfrom
gl3d-blend-rebase
Open

Improve stackgl rendering pipeline for transparent traces#4643
archmoj wants to merge 4 commits into
mainfrom
gl3d-blend-rebase

Conversation

@archmoj

Copy link
Copy Markdown
Contributor

Thanks to fixes made in #4566 and #4480, this PR could fix#3243 as well as #1267.

@alexcjohnson
Commit 27ba6c1 added various failing tests.
Commit efa3cb7 old work done in gl-plot3d rebased and and applied to update the baselines.

…opacityscale
- update volume mock with extreme opacityscale
@archmoj

Copy link
Copy Markdown
ContributorAuthor

Applied #4725 to update conflicting baseline with extremesopacityscale in 3373d8c.

@jackparmer

Copy link
Copy Markdown
Contributor

Stumbled upon some nice work that our old colleague Ricky published recently in this area:
https://observablehq.com/@rreusser/faking-transparency-for-3d-surfaces

@alexcjohnson

Copy link
Copy Markdown
Collaborator

ooh Ricky's demo has some nice features for single-surface presentation - I really like his "cartoon edges" and an option to let the wireframe show through an otherwise opaque surface could be cool. But I don't think this helps us with multiple traces - or a single trace that makes multiple surfaces.

I like the dual depth peeling demo he pointed out - not sure quite what's going on there but it looks promising.

@jackparmer

Copy link
Copy Markdown
Contributor

This issue has been tagged with NEEDS SPON$OR

A community PR for this feature would certainly be welcome, but our experience is deeper features like this are difficult to complete without the Plotly maintainers leading the effort.

Sponsorship range: $15k-$20k

What Sponsorship includes:

  • Completion of this feature to the Sponsor's satisfaction, in a manner coherent with the rest of the Plotly.js library and API
  • Tests for this feature
  • Long-term support (continued support of this feature in the latest version of Plotly.js)
  • Documentation at plotly.com/javascript
  • Possibility of integrating this feature with Plotly Graphing Libraries (Python, R, F#, Julia, MATLAB, etc)
  • Possibility of integrating this feature with Dash
  • Feature announcement on community.plotly.com with shout out to Sponsor (or can remain anonymous)
  • Gratification of advancing the world's most downloaded, interactive scientific graphing libraries (>50M downloads across supported languages)

Please include the link to this issue when contacting us to discuss.

@stevendbrown

Copy link
Copy Markdown

Hi @jackparmer & @archmoj, I'm trying to understand the content of this PR and will be grateful for your help in clarifying what's happening in this PR. It looks like the changes in the PR update or add unit test outputs, but the PR does not include any code to generate those plots, is that right? How were these updated unit test outputs generated if there is no code to generate them?

@archmoj

Copy link
Copy Markdown
ContributorAuthor

Hi @jackparmer & @archmoj, I'm trying to understand the content of this PR and will be grateful for your help in clarifying what's happening in this PR. It looks like the changes in the PR update or add unit test outputs, but the PR does not include any code to generate those plots, is that right? How were these updated unit test outputs generated if there is no code to generate them?

The proposed changes were in gl-plot3d module.
https://github.com/gl-vis/gl-plot3d/compare/gl3d-blend-rebase

@elaude

Copy link
Copy Markdown

Hi, what would be needed to push this over the finish line? My usecase would benefit greatly from correct transparency rendering, so I'd be very happy if this could be fixed. I'm willing to contribute the changes needed myself if there's anything blocking this.

@archmoj

Copy link
Copy Markdown
ContributorAuthor

Hi, what would be needed to push this over the finish line? My usecase would benefit greatly from correct transparency rendering, so I'd be very happy if this could be fixed. I'm willing to contribute the changes needed myself if there's anything blocking this.

@Coding-with-Adam Let's keep track of this for our discussion.
Thanks!

@archmoj

Copy link
Copy Markdown
ContributorAuthor

Hi, what would be needed to push this over the finish line? My usecase would benefit greatly from correct transparency rendering, so I'd be very happy if this could be fixed. I'm willing to contribute the changes needed myself if there's anything blocking this.

As discussed with @gvwilson I'll try to give this another push (after resolving the merge conflicts) and possibly expose this option via a new attribute.

@gvwilsongvwilson removed their assignment Aug 2, 2024
@gvwilsongvwilson added fix fixes something broken P2 considered for next cycle and removed status: has TODOs bug something broken labels Aug 8, 2024
@archmojarchmoj added this to the v3.0.0 milestone Oct 2, 2024
@archmojarchmoj modified the milestones: v3.0.0, v4.0.0Oct 23, 2024
@emilyklemilykl removed this from the v4.0.0 milestone Jun 10, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixfixes something brokenP2considered for next cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Transparent faces in mesh plots overlap incorrectly

7 participants

@archmoj@jackparmer@alexcjohnson@stevendbrown@elaude@gvwilson@emilykl