Implement log_artifact method for live

[karajan1001]

fix: #378

---

# train.py
import time
from pathlib import Path

from dvclive import Live

Path("data").write_text(str(time.time()))
with Live(save_dvc_exp=True) as live:
    live.log_artifact("data")

$ git init
$ dvc init
$ git add train.py
$ git commit -m "init"
$ python train.py
$ git show 6d1a79f
commit 6d1a79fa891e3582a6a9ef93f4e826e5eee5bbe4 (refs/exps/ad/10dbdb6aa0c13b73f48ed8416a65dd57d94899/duddy-yews)
Author: David de la Iglesia Castro <[email protected]>
Date:   Mon Feb 20 15:23:14 2023 +0100

    dvc: commit experiment duddy-yews

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 0000000..3af0ccb
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1 @@
+/data
diff --git a/data.dvc b/data.dvc
new file mode 100644
index 0000000..3088b76
--- /dev/null
+++ b/data.dvc
@@ -0,0 +1,4 @@
+outs:
+- md5: 11d1fcb11076f6c57252cc495d65fb9a
+  size: 17
+  path: data

Outdated

[dberenbaum]

@daavoo Sorry for the potential change in direction here, but I worry it's not very useful to add a .dvc file:

• Inside pipelines, this doesn't make sense.
• Without pipelines, it makes the transition to pipelines harder.

See #468 for an alternative to help with the pipeline transition. We could keep the initial scope narrow to not completely block log_artifact if we can agree on the direction.

[dberenbaum]

LGTM

[dberenbaum]

@daavoo Sorry for the potential change in direction here, but I worry it's not very useful to add a .dvc file:

* Inside pipelines, this doesn't make sense.

* Without pipelines, it makes the transition to pipelines harder.

See #468 for an alternative to help with the pipeline transition. We could keep the initial scope narrow to not completely block log_artifact if we can agree on the direction.

Discussed with @daavoo to not block to wait for #468. Inside pipelines, this can even be used as is if the artifact is not specified as a pipeline output.

I'm trying to help address the transition to pipelines with iterative/dvc#9138.

[codecov-commenter]

Codecov Report

Patch coverage: 92.20% and project coverage change: -0.17 ⚠️

Comparison is base (5b3ffc2) 96.08% compared to head (58075d4) 95.92%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #464      +/-   ##
==========================================
- Coverage   96.08%   95.92%   -0.17%     
==========================================
  Files          38       39       +1     
  Lines        2401     2451      +50     
  Branches      199      203       +4     
==========================================
+ Hits         2307     2351      +44     
- Misses         55       59       +4     
- Partials       39       41       +2     

Impacted Files	Coverage Δ
src/dvclive/live.py	91.51% <68.42%> (-1.69%)	⬇️
tests/conftest.py	93.93% <100.00%> (+5.70%)	⬆️
tests/test_dvc.py	100.00% <100.00%> (ø)
tests/test_log_artifact.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.