include nbformat as a plotly[express] dependency#5380
include nbformat as a plotly[express] dependency#5380afeld wants to merge 1 commit intoplotly:mainfrom
Conversation
Required for MimeType rendering.
|
Hmm, the remaining build error doesn't sense to me… Doesn't seem to be happening on |
|
@afeld AFAIK In general we try to keep Plotly Express dependencies pretty minimal and only require packages which are needed in all scenarios. Would be open to maybe adding a |
Sure. Do you have a sense of how many Plotly Express users are doing so in Jupyter vs. not?
No, I exclusively(?) use Plotly with Jupyter, with build tools like nbconvert and Jupyter Book.
The impression I got was that
That makes sense to me! pip install plotly[express,jupyter]would still be easy for users. I'll update. |
Required for MimeType rendering. Closes #4512.
Code PR
plotly.graph_objects, my modifications concern the code generator and not the generated files.I have added tests or modified existing tests.For a new feature, I have added documentation examples (please see the doc checklist as well).For a new feature or a change in behavior, I have updated the relevant docstrings in the code.