Skip to content

Mplot3d/collection cmap fix #762

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

Merged

Conversation

WeatherGod
Copy link
Member

Colors should no longer disappear while interacting with scatter3d plots

* Found a simple solution to the "colors disappear after interacting with
  a scatter3d plot" bug. Should close matplotlib#761 and close matplotlib#152.
@sb2nov
Copy link

sb2nov commented Mar 13, 2012

So the quality of the image changes after you use that fix. It looks like the alpha gets messed up after that.
I have created two images to show the difference. How do I put those images up ?

@sb2nov
Copy link

sb2nov commented Mar 13, 2012

@WeatherGod : I have mailed you the Images

@WeatherGod
Copy link
Member Author

That isn't a "quality" issue. The question is whether or not the alpha
values should be utilized to indicate depth (which is what the code was
originally supposed to do. In fact, I don't know why your "before" image
doesn't do this. What is your call to scatter?

@sb2nov
Copy link

sb2nov commented Mar 13, 2012

self.ax = self.fig.gca(projection='3d')
self.ax.scatter(xs, ys, zs, c=c, marker='o', s=20)

@WeatherGod
Copy link
Member Author

ok... in any case, the behavior you had in the past is wrong and it appears that my patch inadvertently fixed it. Perhaps a kwarg could be passed into scatter() to force it to not modify the alpha values?

@sb2nov
Copy link

sb2nov commented Mar 14, 2012

Cool. Thanks a lot for your help.

WeatherGod added a commit that referenced this pull request Mar 17, 2012
@WeatherGod WeatherGod merged commit 07929d4 into matplotlib:v1.1.x Mar 17, 2012
@WeatherGod WeatherGod deleted the mplot3d/collection_cmap_fix branch July 23, 2014 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants