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

log_image support? #125

Open
elgalu opened this issue Sep 1, 2021 · 3 comments
Open

log_image support? #125

elgalu opened this issue Sep 1, 2021 · 3 comments

Comments

@elgalu
Copy link

elgalu commented Sep 1, 2021

Similar to https://www.tensorflow.org/api_docs/python/tf/summary/image or https://mlflow.org/docs/latest/python_api/mlflow.tracking.html#mlflow.tracking.MlflowClient.log_image

@guysmoilov
Copy link
Member

Thanks @elgalu , that is a suggestion we've been thinking about how to implement properly.
Why not just save the image as part of the commit, next to the output metrics.csv and params.yml?
You expect to see the image as part of the experiment dashboard in the end?

@elgalu
Copy link
Author

elgalu commented Sep 1, 2021

You expect to see the image as part of the experiment dashboard in the end?

Yes, even more, we expect to see each logged image in each step to track model evolution

@guysmoilov
Copy link
Member

Interesting, I would be happy to hear more if you can share.

  • Do you have an example of a different tool that does as a reference?
  • Can you describe the use case? Why is this valuable to you and how you expect to use this? Details would be valuable, e.g. if you only actually look at 2-3 images and not for every step, or only once every epoch vs. once every step
  • Can you describe how you imagine using this feature in the UI?

In the meantime, why not save images to a DVC tracked folder with the step index as part of the image name, as a workaround?

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

No branches or pull requests

2 participants