-
Notifications
You must be signed in to change notification settings - Fork 38
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_plot: allow for y
to be a list
#549
Comments
+1 for this. I came here looking for a way to log a plot with a definition like
Is this possible to write directly to data.json instead of |
Yes. You can always manually write to json and configure dvc.yaml yourself. See https://dvc.org/doc/user-guide/experiment-management/visualizing-plots for a guide on how to do it. |
Thanks. I was able to add a plot like this to the |
Yes, live updates are limited to dvclive plots. You should be able to still see the manual plots once the experiment completes if you run |
I see. Thanks for the clarification. I think that should be fine, but would be nice to get live updates. It's just not what I expected. |
@jenfredwell Have you actually tried it? I think it will work even though it's not documented. See this example: import pandas as pd
from dvclive import Live
from sklearn.datasets import load_iris
iris = load_iris()
datapoints = pd.DataFrame(data=iris.data, columns=iris.feature_names)
with Live() as live:
live.log_plot(
"sepal",
datapoints,
x="sepal length (cm)",
y=["sepal width (cm)", "petal width (cm)"],
template="scatter",
title="Sepal length vs Sepal/petal width"
) This renders a plot like: If you confirm it works for you, we can mention support in the docs as well. |
Yes, this actually does work but doesn't seem to update in real time. It looks like the |
Sorry @jenfredwell, I had forgotten that Studio does not support live updates of |
Ok good to know. Is this in the roadmap by chance? It would be helpful to us to consolidate some of what we're starting to track with dvclive. |
Sorry, it's not in the roadmap at the moment |
Should we allow for
y
to be a list?Edit: can be a follow up, so not blocking, but I think it makes it significantly more useful.
Originally posted by @dberenbaum in #543 (comment)
The text was updated successfully, but these errors were encountered: