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

Doc fixes and removal of ContinuousColorScale #779

Merged
merged 4 commits into from
Jan 8, 2016

Conversation

simleb
Copy link
Contributor

@simleb simleb commented Dec 30, 2015

I implemented the changes around ContinuousColorScale and color_continuous discussed in #763. I took the opportunity to fix a bunch of typos and update a few things in the docs as well.

I don't know how but the docs will need to be regenerated at some point. I suppose only @dcjones can do that.

I chose colormap as the new keyword for the function that maps values in [0,1] to a color (my Matlab background is showing here, sorry). The name is of course open to discussion.

Should this file simply be removed?
For consistency with x_continuous, y_continuous.
Note that ContinuousColorScale is still in Gadfly
but no longer in the documentation.

This introduces a new “colormap” keyword for
color_continuous to replace ContinuousColorScale's
“f” argument.
@timholy
Copy link
Collaborator

timholy commented Jan 8, 2016

Wow, that's a lot of doc fixes. Thanks!

timholy added a commit that referenced this pull request Jan 8, 2016
Doc fixes and removal of ContinuousColorScale
@timholy timholy merged commit 51e245d into GiovineItalia:master Jan 8, 2016
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.

None yet

2 participants