Skip to content

background color not showing in 3DMesh Plots #929

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
Kully opened this issue Sep 12, 2016 · 6 comments
Closed

background color not showing in 3DMesh Plots #929

Kully opened this issue Sep 12, 2016 · 6 comments
Labels
bug something broken

Comments

@Kully
Copy link
Contributor

Kully commented Sep 12, 2016

@etpinard
I noticed that the background colors for Trisurfs in Python were not their usual grey. I found a similar issue in the JS tutorial for 3D Axes and recreated the same problem in Codepen by forcing a background color of rgb(255,0,0) for illustrative purposes.

@etpinard
Copy link
Contributor

Thanks. This has been broken since the 1.17.0 release.

@etpinard etpinard added the bug something broken label Sep 12, 2016
@etpinard
Copy link
Contributor

etpinard commented Sep 12, 2016

Oh. This is really unfortunate.

Looks like stackgl/gl-shader#16 is causing this issue.

Reverting back to [email protected] fixes the issue. I also noticed that some tick labels aren't showing on 3D graphs on 1.17.x, reverting to [email protected] also fixes that.

The worse part is that npm run test-image -- gl3d_* fails locally on master but CiricleCI fail to catch the problem - most likely it didn't properly update gl-shader to 4.2.1 during #850.

cc @monfera

@etpinard
Copy link
Contributor

... maybe we should start committing an npm-shrinkwrap.json file to ensure that CircleCi run npm i when updating nested dependencies.

@etpinard
Copy link
Contributor

etpinard commented Sep 12, 2016

I'm going to make a patch release after #931 is merged.

@Kully this issue will be fixed in v1.17.2

@Kully
Copy link
Contributor Author

Kully commented Sep 12, 2016

@Kully this issue will be fixed in v1.17.2

👍

@etpinard
Copy link
Contributor

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

No branches or pull requests

2 participants