Skip to content

Rasterize colorbar when it has many colors; closes #4480 #4481

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
merged 1 commit into from
Jun 8, 2015

Conversation

efiring
Copy link
Member

@efiring efiring commented May 27, 2015

I think this is the simplest possible way to reduce the number of complaints about the appearance of the colorbar as rendered from vector files. It could be made more complicated by using an rcParam for the threshold, and/or by adding a kwarg, but I'm not convinced that either of these would be worthwhile.

@efiring
Copy link
Member Author

efiring commented May 27, 2015

Travis failures are unrelated.

@tacaswell
Copy link
Member

How about making the threshold a public attribute of Colorbar? If we hard code this someone will complain.

@tacaswell tacaswell added this to the next point release milestone May 28, 2015
@efiring
Copy link
Member Author

efiring commented May 28, 2015

Yes, I know... It sounds like you are talking about making it a class attribute for now, in anticipation of all instance configuration variables ending up declared at the class level as Configurables or some such. Correct?

@tacaswell
Copy link
Member

I was thinking as an instance level attribute, but class level makes more sense.

@efiring
Copy link
Member Author

efiring commented Jun 7, 2015

@tacaswell If this looks OK, I can add a what's new entry. @WeatherGod, comments?

@tacaswell
Copy link
Member

👍 from me.

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

Successfully merging this pull request may close these issues.

3 participants