Simon Høxbro Hansen

Results 143 comments of Simon Høxbro Hansen

The `register_cmap` function has been in colorcet as a "normal" matplotlib import for a long time, and the updated `register_cmap` function added in https://github.com/holoviz/colorcet/pull/90 should be backward compatible. So I...

This seems to happen because of a race condition happening between `len(store_ids)` and `max(store_ids)`. It could be solved by converting `store_ids` to a list in the above line. Thoughts @jlstevens?...

I noticed that an empty `opts` will reset previous options, this should not be the case in my opinion. ![image](https://user-images.githubusercontent.com/19758978/198291991-06ee7566-ab0b-4197-959b-ebcc9333a0e7.png)

Thank you for the suggestion. As you have noted, we do not have the logic to infer the legend from the dimension expression. I will mark this as a feature...

> Funny that the fix for deprecated _ipython_display_ is inside a function of the same name :) Yeah, the irony was not lost on me either.

No worries, nothing urgent in getting this merged.

I can reproduce the problem. And looking at the code, it is easy to see why it happens, as there is no uniqueness to the name of a class. The...

Yes. This should be fixed when this https://github.com/holoviz/holoviews/pull/5470 is merged. Curious about why you see the warning in your notebook. Do you have any custom settings enabled?

Looking at the code here: https://github.com/holoviz/hvplot/blob/17ce0cc18a0393ae82d3a9f6f11b3bc4cad29b30/hvplot/converter.py#L1623-L1637 It seems to not be supported when y is a list or tuple, so making by making ´y´ a string "fixes" it. ![image](https://user-images.githubusercontent.com/19758978/186937178-60a85404-4fdc-4e22-a391-631d32ca3af4.png) A...

1. I think it is fair to work in this case or raise an error that `by` does not work when `y` is a list/tuple. 2. See the answer above....