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

Indicate color clipping and control clipping colors #686

Merged
merged 6 commits into from
Oct 12, 2016
Merged

Conversation

philippjfr
Copy link
Member

@philippjfr philippjfr commented May 12, 2016

As the title says, adds a plot option to extend the colorbar in a particular. Here's an example:

%%opts Image [colorbar=True]
hv.Layout([hv.Image(np.random.rand(10,10))(plot=dict(cbar_extend=opt))
           for opt in ['min', 'max', 'both']])

image

Can also consider automatically enabling this if your data falls outside the specified range but that's another discussion.

@jbednar
Copy link
Member

jbednar commented May 12, 2016

I don't get it -- what does it mean to extend the colorbar? It doesn't seem extended, it just now has a pointy end. Do you mean, make the colorbar visually indicate that it extends beyond the area shown, without actually changing the colorbar or the colormapping? If so that's not what the option name or the name of this issue suggests, and seems very confusing.

@philippjfr
Copy link
Member Author

True, the name isn't great, it's what matplotlib uses.

@philippjfr
Copy link
Member Author

philippjfr commented May 12, 2016

This would also be more useful with an option to supply different colors for values that are above and below the range (and for NaN values). We can brainstorm about a nice way to specify that.

@jlstevens
Copy link
Contributor

pointy_cbar sounds more descriptive to me! ;-p

@jbednar
Copy link
Member

jbednar commented May 12, 2016

I'm not sure it makes sense to provide direct control over this option, in HoloViews. Seems like HoloViews should just automatically turn it on if the user has selected a colormap that ends up clipping at a particular end of the colorbar, so that the visual representation is a reliable indicator of something meaningful. As it is, there's no relation between whether the user has explicitly enabled a pointy colorbar and anything about the actual plot, so indeed Jean-Luc's suggestion of calling it pointy_cbar is accurate!

@jlstevens
Copy link
Contributor

Using this to indicate clipping when clipping occurs does make a lot more sense. We would still need a parameter to turn this behaviour on and off though...

@jlstevens
Copy link
Contributor

Also I assume one problem this this suggestion is we might not know if clipping has occurred without inspecting the data range first? Sounds tricky...you might have to compare the data min/max with the limits to know if it should be pointy or not.

@philippjfr
Copy link
Member Author

philippjfr commented May 12, 2016

Sounds tricky...you might have to compare the data min/max with the limits to know if it should be pointy or not.

It might be fairly straightforward and the overhead wouldn't be huge, so might be worth doing. Coupled with a way to specify special colors for min, max and NaN values that would be a nice addition.

@jlstevens
Copy link
Contributor

It might be fairly straightforward and the overhead wouldn't be huge..

Sure. As long as there is a way to turn it off and keep the current behaviour in case it ever becomes a problem. I agree it would be a nice feature and I'm wondering if there might be a way to specify the NaN values and clipping colors at the same time...

@jbednar
Copy link
Member

jbednar commented May 12, 2016

That makes sense: the user can optionally enable having pointy cbars indicate such clipping. That way, whenever there is such a pointy cbar, it does accurately reflect something about the current plot, rather than just being some meaningless style option. Yet those who don't ever want pointy cbars can turn them off.

@philippjfr philippjfr changed the title Added plot option to extend the colorbar in matplotlib Indicate color clipping and control clipping colors in matplotlib May 13, 2016
@philippjfr
Copy link
Member Author

So since I actually needed this for some plots of mine I've gone ahead and implemented it. If you've got a better suggestion for the clipping_colors parameter please suggest it and tell me and I also still need a name for the option that turns the clipping indicators on and off in general.

@jlstevens
Copy link
Contributor

Apparently ('w', 1) is used because the colorbar code in matplotlib takes the color and a separate alpha. I would much prefer to pretend matplotlib is consistent and support standard matplotlib colors, including the RGBA tuples.

@philippjfr
Copy link
Member Author

Sure, shouldn't be hard.

@@ -497,8 +497,10 @@ class ColorbarPlot(ElementPlot):
colorbar = param.Boolean(default=False, doc="""
Whether to draw a colorbar.""")

cbar_width = param.Number(default=0.05, doc="""
Width of the colorbar as a fraction of the main plot""")
clipping_colors = param.Dict(default={'NaN': ('w', 1)}, doc="""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand what I expect to happen to NaN values but an example of how to set the min and max colors would be nice (in the docstring).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed.

@jlstevens
Copy link
Contributor

Philipp tells me you would specify a max clipping like this:

{'min': 'r', 'Nan': 'w'}

Would setting these colors make the colorbar pointy as appropriate (if the data is clipped?). If so wouldn't an empty dictionary correspond to the current behaviour without pointy colorbars?

@philippjfr
Copy link
Member Author

philippjfr commented May 13, 2016

If so wouldn't an empty dictionary correspond to the current behaviour without pointy colorbars?

No, not specifying a special color just sets the min and max color in the cmap as the clipping color, as in the example above.

@philippjfr philippjfr force-pushed the cbar_extend branch 2 times, most recently from 25945f8 to 1bc85cf Compare May 13, 2016 10:45
@jlstevens
Copy link
Contributor

So why not have something like:

cbar_extension = {'min': 'default', 'max':'default'}

To indicate pointy colorbars with the min/max of the colormap where you can then specify your own custom colors instead of 'default'?

@philippjfr
Copy link
Member Author

Sounds good, if we can come up with a better name for the parameter.

@jbednar
Copy link
Member

jbednar commented May 13, 2016

Maybe cbar_clip?

@jlstevens
Copy link
Contributor

Given that colorbar is a parameter name, colorbar_clip would appear next to it. For that reason I might like the longer name.

@jbednar
Copy link
Member

jbednar commented May 13, 2016

Sure, if "clip" is accurate as a description about what it does (which I've somewhat lost track of by this point. :-)

@philippjfr
Copy link
Member Author

philippjfr commented May 13, 2016

We already have cbar_width and cbar_padding so it should at least be consistent.

Edit: Of course we could decide to rename those as well.

@jlstevens
Copy link
Contributor

We already have cbar_width and cbar_padding so it should at least be consistent.

In that case we should alias colorbar with cbar to enable/disable color bars.

@philippjfr
Copy link
Member Author

philippjfr commented May 13, 2016

I'd suggest just calling it clipping_colors or color_clipping. Adding the pointy bit to the color bar is just a secondary side effect. Even without the colorbar enabled this will control the out-of-range colors so it's not really a colorbar option.

Edit: I guess why I was hesitant about the {'max': 'default'} suggestion is for the same reason, this parameter doesn't necessarily have anything to do with the colorbar, so enabling it that way feels a bit weird.

@jlstevens
Copy link
Contributor

clipping_colors seems fine. So could you still use it to disable pointy colorbars?

@jbednar
Copy link
Member

jbednar commented May 24, 2016

See #689 for proposed UI.

@philippjfr
Copy link
Member Author

Ready to merge once clipping color support is fixed in bokeh: bokeh/bokeh#5323

@philippjfr philippjfr changed the title Indicate color clipping and control clipping colors in matplotlib Indicate color clipping and control clipping colors Oct 7, 2016
@@ -748,6 +748,11 @@ class ColorbarPlot(ElementPlot):
location, orientation, height, width, scale_alpha, title, title_props,
margin, padding, background_fill_color and more.""")

clipping_colors = param.Dict(default={'NaN': (0, 0, 0, 1)}, doc="""
Dictionary to specify colors for clipped values, allows setting
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RGBA tuples are fine though are any other color specifications allowed? E.g hex strings?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, docstring should be updated to reflect that.

@jlstevens
Copy link
Contributor

Looks good. Happy to merge when the tests pass.

@jlstevens
Copy link
Contributor

Tests have all passed. Merging.

@jlstevens jlstevens merged commit 3c29b0b into master Oct 12, 2016
@philippjfr philippjfr deleted the cbar_extend branch October 14, 2016 01:55
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants