search

plotly / plotly.js

Open-source JavaScript charting library behind Plotly and Dash
https://plotly.com/javascript/
MIT License
16.97k stars 1.86k forks source link

Use surfacecolor, if defined, instead of z to color the contour lines #4097

Closed kapraran closed 3 months ago

kapraran commented 5 years ago

I suggest that when usecolormap in surface.contours is true, the lines should be colored using the surfacecolor, if defined, otherwise it will use the colorscale as a fallback.

Demo

As you can see in the demo, i'd like the countours in the second scene to use the surfacecolor as a source for coloring instead of the default colorscale that they currently use.

Thanks in advance!

etpinard commented 5 years ago

I'd call that a bug. Thanks for reporting!

etpinard commented 5 years ago

Assigning @archmoj

archmoj commented 4 years ago

Not sure if this is a bug. What is the issue here?

gpdonnarumma commented 4 years ago

If is possible to get similar result on a surface in 3D space. As you can see in the image the contours are based on the color area (surfacecolor) not axes data IMG-20191126-WA0007

gpdonnarumma commented 4 years ago

Hi, sorry for disturbing you.

Is there any news about this feature? Is possible to use "surfacecolor" data and not axis value to calculate contour line?

archmoj commented 4 years ago

If is possible to get similar result on a surface in 3D space. As you can see in the image the contours are based on the color area (surfacecolor) not axes data IMG-20191126-WA0007

@gpdonnarumma Is the image posted above relevant to this ticket?

gpdonnarumma commented 4 years ago

If is possible to get similar result on a surface in 3D space. As you can see in the image the contours are based on the color area (surfacecolor) not axes data IMG-20191126-WA0007

@gpdonnarumma Is the image posted above relevant to this ticket?

Sorry, I think it's not relevant. You can remove it.

archmoj commented 4 years ago

@gpdonnarumma GitHub today had some down times. And I am not sure if you noticed my previous message. Anyway I post it here again: Until the bug is fixed one may consider using opacityscale feature as illustrated in this demo.

gpdonnarumma commented 4 years ago

Hi, thank you for your help. Opacityscale is not what I need... but I think that is a great feature. I'll use it in other functions.

So I'm working on how to realize something like a contour series over a surface3D serie.

@gpdonnarumma@gmail.com

On Fri, May 22, 2020 at 11:21 PM Mojtaba Samimi notifications@github.com wrote:

@gpdonnarumma https://github.com/gpdonnarumma GitHub today had some down times. And I am not sure if you noticed my previous message. Anyway I post it here again: Until the bug is fixed one may consider using opacityscale feature as illustrated in this demo https://codepen.io/MojtabaSamimi/pen/QWjYRVV?editors=0010.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/plotly/plotly.js/issues/4097#issuecomment-632913328, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGENNLPDTIQ6XRFXCU3AHVTRS3UHLANCNFSM4IIMDTBQ .

jackparmer commented 4 years ago

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: $5k-$10k

What Sponsorship includes:

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

gvwilson commented 3 months ago

Hi - we are trying to tidy up the stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for several years, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. Thanks for your help - @gvwilson