-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add performance doc #4769
Add performance doc #4769
Conversation
@marthacryan @archmoj could I get your feedback on this first draft? |
doc/python/b64.md
Outdated
|
||
## Unsupported Attributes | ||
|
||
Arrays passsed to attributes with the following names do not use the Plotly.js base64 typed arrays functionality: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we should even bring up base64 here. Maybe it could just say something like "performance improvements related to using pandas and numpy arrays"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Along the same lines, maybe we could rename this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. That info probably isn't that useful for the user. And I'll rename the file too. Thanks @marthacryan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made those updates. There's still a mention here. Is it base64-encoded typed arrays support in Plotly.js specifically that allows for the improved performance?
Plotly.py uses Plotly.js for rendering, which supports base64-encoded typed arrays. In Plotly.py, NumPy array and NumPy-convertible arrays are base64 encoded before being passed to Plotly.js for rendering.
Co-authored-by: Nathan Drezner <38958867+ndrezn@users.noreply.github.com>
@marthacryan do you have any additional feedback on this one? |
Maybe we should rename the file as well to be called performance. Also, it was sort of hard to find the numpy section because the webgl part was so long. I wonder if we could split them into separate pages that are both linked from the performance page? Or is there some kind of table of contents or link that could be added to the top of the doc to make it easy to jump to the section on numpy arrays? |
Thanks @marthacryan. When it is in the docs, there will be sidebar with the TOC: It's a good point about the Webgl section though. Some of the examples I think could go. Taking a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be great to combine https://plotly.com/python/datashader/ into this page too!
Documentation PR
doc/README.md
filedoc-prod
branch OR it targets themaster
branchpx
example if at all possibleplotly.graph_objects as go
/plotly.express as px
/plotly.io as pio
df
fig = <something>
call is high up in each new/modified example (eitherpx.<something>
ormake_subplots
orgo.Figure
)fig.add_*
andfig.update_*
rather thango.Figure(data=..., layout=...)
in every new/modified examplefig.add_shape
andfig.update_xaxes
are used instead of bigfig.update_layout
calls in every new/modified examplefig.show()
is at the end of each new/modified exampleplotly.plot()
andplotly.iplot()
are not used in any new/modified exampleCode PR
plotly.graph_objects
, my modifications concern thecodegen
files and not generated files.modified existing tests.
new tutorial notebook (please see the doc checklist as well).
-->