Skip to content
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

add colorbars to density plots #9150

Open
jasongrout opened this issue Jun 5, 2010 · 3 comments · May be fixed by #35343
Open

add colorbars to density plots #9150

jasongrout opened this issue Jun 5, 2010 · 3 comments · May be fixed by #35343

Comments

@jasongrout
Copy link
Member

See #8368 for the contour plot patch; probably much of it can be recycled for density plots.

CC: @kcrisman

Component: graphics

Issue created by migration from https://trac.sagemath.org/ticket/9150

@jasongrout
Copy link
Member Author

comment:1

Kirill Vankov (on sage-devel just now) also asked that colorbars be added as an option to matrix plots as well, which would be straightforward once they were added to density plots.

@kcrisman
Copy link
Member

comment:2

This ask.sagemath.org question asks for exactly the same thing, and user "Shashank" has a nice mpl example of it.

@ppurka
Copy link
Member

ppurka commented Oct 28, 2012

comment:3

The Colorbar class in mpl is a bit strange. It doesn't take in all the options for all kinds of figures. The way it is used in contour_plot in Sage works only for that case.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
@balejk balejk linked a pull request Mar 24, 2023 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants