Skip to content

surf() not working correctly #260

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.

Sign up for GitHub

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

Jump to bottom
Closed
xarico10 opened this issue Aug 2, 2021 · 2 comments
Closed

surf() not working correctly #260

xarico10 opened this issue Aug 2, 2021 · 2 comments
Assignees
@gilbertogalvis

Comments

@xarico10
Copy link
Contributor

xarico10 commented Aug 2, 2021

When using surfl() to creates a three-dimensional surface plot with highlights from a light source, such light effect isn't working.

[X,Y] = meshgrid(1:0.5:10,1:20);
Z = sin(X) + cos(Y);
sl = surfl(X,Y,Z,'light');

fig2plotly(gcf, 'offline', false);

Expected and generated (https://chart-studio.plotly.com/~xarico10/761/#/) plots:
Captura de Pantalla 2021-08-02 a la(s) 10 48 19 a  m
Captura de Pantalla 2021-08-02 a la(s) 10 48 01 a  m

Examples on using surfl() can be found on:
https://www.mathworks.com/help/matlab/ref/surfl.html
@jackparmer
Copy link
Contributor

We should use the same Colorscale as well

@xarico10 xarico10 mentioned this issue Aug 2, 2021
Open
@3alimoh 3alimoh self-assigned this Aug 4, 2021
@jackparmer jackparmer assigned gilbertogalvis and unassigned 3alimoh Aug 5, 2021
gilbertogalvis pushed a commit that referenced this issue Aug 5, 2021
@galvisgilberto
fix issue #260: surface
0eb86ae
@gilbertogalvis
Copy link
Contributor

Screen Shot 2021-08-05 at 6 56 21 PM

I think this is fixed

xarico10 added a commit that referenced this issue Aug 8, 2021
@xarico10
Merge pull request #316 from plotly/issue260
39eeaaf 
fix issue #260: surface
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gilbertogalvis gilbertogalvis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jackparmer @gilbertogalvis @xarico10 @3alimoh