Skip to content
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

dvclive: Document image support #3013

Closed
daavoo opened this issue Nov 9, 2021 · 3 comments · Fixed by #3207
Closed

dvclive: Document image support #3013

daavoo opened this issue Nov 9, 2021 · 3 comments · Fixed by #3207
Assignees
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: dvclive Content of /doc/dvclive

Comments

@daavoo
Copy link
Contributor

daavoo commented Nov 9, 2021

Version 0.4.3 of dvclive added support to logging images, following dvc support for it.

@daavoo daavoo added team/dvclive C: dvclive Content of /doc/dvclive labels Nov 9, 2021
@daavoo
Copy link
Contributor Author

daavoo commented Nov 9, 2021

I was considering waiting for #2956 to have a consistent narrative

@daavoo daavoo self-assigned this Nov 9, 2021
@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Nov 11, 2021

Looks like there's consensus in #2956 (comment). Only call scalars metrics. Plots can be different things (plots data files and the corresponding plot visualization DVC can generate vs image files that DVC can also render). Does that help?

@shcheklein shcheklein added A: docs Area: user documentation (gatsby-theme-iterative) and removed A: dvclive labels Dec 31, 2021
@dberenbaum
Copy link
Collaborator

@daavoo Is that enough info to get started on this? I don't think you should wait for #2956 since rewording those docs is not as high priority as documenting this new feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: dvclive Content of /doc/dvclive
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants