Simon Høxbro Hansen

Results 461 comments of Simon Høxbro Hansen

I'm not opposed to this PR, but currently, it breaks CI. Did you try to pass in limits to `hv.dim("Magnitude").norm(limits=(0, None))`? Or would you like it to be dynamically updated?...

Thank you for the PR, this will be part of the next release 1.21.0

I would also expect this to work, so I will mark this as a bug. I've been playing around with it myself. Without a fix, I think the "cleanest" way...

I can see we need to update HoloViews at least in one place: https://github.com/holoviz/holoviews/blob/eb79ab52414f53b5990f2b624edd4ba582a3fa6e/holoviews/operation/datashader.py#L5

I can't seem to recreate the following: ![image](https://github.com/user-attachments/assets/b4b92c6f-7b2a-469e-99e4-e4ee17140c71) What are your versions? Can you try again and see if you still see the problem or paste in the output of...

Pure Bokeh example, when something like this is cropped, it is usually not even a holoviews problem. @mattpap, I think this will fall under the same underlying problem as seen...

Yeah was in a hurry meant https://github.com/holoviz/holoviews/issues/5438 because of frame_* and layout, but not sure anymore 🙃

This is because we go down a bad route because of the legacy `color_index`. Setting `color_index` to 5 in opts will give the desired behavior but raise a warning. https://github.com/holoviz/holoviews/blob/e818c4d7301838301b98f21103649cb93505be8c/holoviews/plotting/bokeh/path.py#L259-L261...

Honestly, I'm not sure if we should support it. The convenience of `hover_formatters` is that we don't need to do a Bokeh import, but if you are already doing that,...

Looks much better! However, one test is failing. Can you take a look at it before I review it?