Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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
igiagkiozis merged 1 commit into plotly:dev from foresterre:patch-impl-write
Jan 1, 2022
Merged

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

igiagkiozis merged 1 commit into plotly:dev from foresterre:patch-impl-write
Jan 1, 2022

Conversation

@foresterre
Copy link
Contributor

@foresterre foresterre commented Jul 15, 2021
edited
Loading

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
Copy link
Contributor Author

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

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /