Skip to content

Commit

Permalink
test for tracking dvcyaml (#713)
Browse files Browse the repository at this point in the history
* live: Set new defaults `report=None` and `save_dvc_exp=True`.

* frameworks: Drop model_file.

* update examples

* Write to root dvc.yaml (#687)

* add dvcyaml to root

* clean up dvcyaml implementation

* fix existing tests

* add new tests

* add unit tests for updating dvcyaml

* use posix paths

* don't resolve symlinks

* drop entire dvclive dir on cleanup

* fix studio tests

* revert cleanup changes

* unify rel_path util func

* cleanup test

* refactor tests

* add test for multiple dvclive instances

* put dvc_file logic into _init_dvc_file

---------

Co-authored-by: daavoo <[email protected]>

* report: Drop "auto" logic.

Fallback to `None` when conditions are not met for other types.

* studio: Extract `post_to_studio` and decoulple from `make_report` (#705)

* refactor(tests): Split `test_main` into separate files.

Rename test_frameworks to frameworks.

* fix matplotlib warning

* drop dvc.yaml prefix from studio plots

* refactor

* drop .dvc files

* track dvcyaml file

* Studio dvcyaml (#709)

* drop dvc.yaml prefix from studio plots

* refactor

* drop .dvc files

* dvc version bump

* drop unneeded relpath call

* drop extra make_report call

* test for git add dvc.yaml in exp run

---------

Co-authored-by: daavoo <[email protected]>
  • Loading branch information
Dave Berenbaum and daavoo authored Sep 21, 2023
1 parent 3524a65 commit 3a2e850
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/test_dvc.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ def test_untracked_dvclive_files_inside_dvc_exp_run_are_added(
live.log_metric("foo", 1)
live.next_step()
live._dvc_repo.scm.add.assert_any_call(["dvclive/metrics.json", plot_file])
live._dvc_repo.scm.add.assert_any_call(live.dvc_file)


def test_dvc_outs_are_not_added(tmp_dir, mocked_dvc_repo, monkeypatch):
Expand Down

0 comments on commit 3a2e850

Please sign in to comment.