Skip to content

Color argument not working on surface plots #258

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

Closed
xarico10 opened this issue Aug 1, 2021 · 1 comment · Fixed by #289
Closed

Color argument not working on surface plots #258

xarico10 opened this issue Aug 1, 2021 · 1 comment · Fixed by #289
Assignees
Labels
⭐⭐⭐ multiplier Issues that could close multiple related issues at once

Comments

@xarico10
Copy link
Contributor

xarico10 commented Aug 1, 2021

When using the input argument C to specify a color map for a surface plot, the color isn't being changed.

[X,Y] = meshgrid(1:0.5:10,1:20);
Z = sin(X) + cos(Y);
C = X.*Y;
surf(X,Y,Z,C)

fig2plotly(gcf, 'offline', false);

Expected and generated (https://chart-studio.plotly.com/~xarico10/753/#/) plots:
Captura de Pantalla 2021-07-31 a la(s) 7 03 08 p  m
Captura de Pantalla 2021-07-31 a la(s) 7 03 39 p  m

Examples on using C input argument can be found on:
https://www.mathworks.com/help/matlab/ref/surf.html#bvgs7ws-1

@xarico10 xarico10 pinned this issue Aug 2, 2021
@xarico10 xarico10 unpinned this issue Aug 2, 2021
@xarico10 xarico10 added the ⭐⭐⭐ multiplier Issues that could close multiple related issues at once label Aug 2, 2021
@3alimoh 3alimoh self-assigned this Aug 2, 2021
@3alimoh
Copy link
Contributor

3alimoh commented Aug 3, 2021

Hello I think this issue will be closed very soon.
I could apply the color argument but with the standard colors of the plotly which refers to issue #255
But this one I think is about to close.

3alimoh added a commit to 3alimoh/plotly-graphing-library-for-matlab that referenced this issue Aug 3, 2021
This will fix the issue plotly#258
@3alimoh 3alimoh linked a pull request Aug 4, 2021 that will close this issue
3alimoh added a commit that referenced this issue Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐⭐⭐ multiplier Issues that could close multiple related issues at once
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants