Skip to content

Add a method to render the plot to anything implementing Write #44

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

Merged
merged 1 commit into from
Jan 1, 2022
Merged

Add a method to render the plot to anything implementing Write #44

merged 1 commit into from
Jan 1, 2022

Conversation

foresterre
Copy link
Contributor

@foresterre foresterre commented Jul 15, 2021

Plotly supported rendering the plot to HTML, and writing the output to a file, located at a user given path. Often however, it's desirable to split the process of rendering and the process of outputting the rendering. In addition, writing to a path is not always the most flexible, since you don't always have a file path. It can then be useful to be able to write the rendered entity to any writable buffer, which can be accomplished by using the std::io::Write trait instead of a path, which is what this change set does.

If desirable, I can also rework it to support the other output formats.

Plotly supported rendering the plot to HTML, and writing the output to a file, located at a user given path. Often however, it's desirable to split the process of rendering and the process of outputting the rendering. In addition, writing to a path is not always the most flexible, since you don't always have a file path. It can then be useful to be able to write the rendered entity to any writable buffer, which can be accomplished by using the std::io::Write trait instead of a path, which is what this change set does.
@foresterre foresterre changed the title add matches to Axis Add a method to render the plot to anything implementing Write #40 by foresterre was closed 3 hours ago updated 3 hours ago Jul 15, 2021
@foresterre foresterre changed the title Add a method to render the plot to anything implementing Write #40 by foresterre was closed 3 hours ago updated 3 hours ago Add a method to render the plot to anything implementing Write Jul 15, 2021
@foresterre
Copy link
Contributor Author

Follow up to #40, now against the (hopefully 😄 ) correct branch.

@igiagkiozis
Copy link
Contributor

Thanks for the update @foresterre; will have a closer look this weekend.

@igiagkiozis igiagkiozis merged commit 60669d3 into plotly:dev Jan 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants