You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Just looked at the relevant source code - it's fairly easy to add. Would you accept a pull request?
The interesting question then becomes how to handle the stuff between subsequent lines 731 and 744. Ignore auto_open == True and include_plotlyjs == 'directory' options or raise exceptions if filename is an open file handle?
Hi @s-m-e, thanks for the suggestion. Yes, and pull request would be appreciated! It's fine to raise exceptions for combinations of parameters that don't make sense together.
In case you're interested in how I'd like to see this API refactored in the future, see #1098.
Hi - we are currently trying to tidy up Plotly's public repositories to help us focus our efforts on things that will help users most. Since this issue has been sitting for several years, I'm going to close it. If it's still a concern, we'd be grateful if you could open a new issue (with a short reproducible example if appropriate) so that we can add it to our backlog. Thanks for your help - @gvwilson
Currently, the
filename
parameter ofplot
only supports filenames (temp-plot.html
is default). It does not support open file handles.The latter is very useful when writing e.g. command line tools with click (or similar tools), which handles opening files specified by a user on the command line transparently and only provides a file handle to the script.
The text was updated successfully, but these errors were encountered: