New subplot foundation for Plotly backend #3255
This PR introduces a new foundation for combining plotly elements into a figure.
Currently, the plotly backend makes use of the
There is a pretty detailed commit log, but here are the two top-level goals:
This gives us more freedom in the low-level figure manipulation utilities (those added in `holoviews/plotting/plotly/util.py`) and it is somewhat faster as well
…ring This way we still get full property validation, but it only happens once at the very end.
This will trigger property validation errors during testing if any of the backend operations produce invalid plotly figure dicts
I'd be happy to merge this PR as is, thanks for the detailed docstrings! I'll probably have to play around a bit to get a good feel for the way the figure merge, subplot merging and figure grid code works but this seems like a very solid foundation to build on.
One thing I'd be happy to do at this point is to remove the big warning about the experimental nature of the backend. I may also work on porting the new so called
First of all, thank you very much @jonmmease for your work here!
Here is the current warning:
I would like to remove this warning but I don't think we would need to debate it at all if @jonmmease were willing to volunteer as the maintainer for the plotly support. It probably wouldn't have to involve all the responsibilities listed above (e.g docs and responding to all user issues) but having someone willing to own the code would definitely help a lot.
Alternatively, we could keep the warning but edit it so it if softer (e.g still asking for help but not saying it is an experimental and unsupported feature).
The reason for the warning is that we aren't using the Plotly backend for our own projects at this time, and so its maintenance can't be folded into anything we're working on. I'd be happy to remove the warning as long as someone were looking out for it even a little bit...
Hi @philippjfr @jlstevens @jbednar ,