Skip to content

Conversation

@daavoo
Copy link
Contributor

@daavoo daavoo commented Apr 4, 2023

Only if they are currently untracked.

Closes #7842

@daavoo daavoo force-pushed the 7842-dvc-exp-run-queue-deleting-file-paramsyaml-for-each-command-call branch from 58ecdf3 to 6fdb7f5 Compare April 4, 2023 17:20
@daavoo daavoo self-assigned this Apr 4, 2023
@daavoo daavoo added A: experiments Related to dvc exp bugfix fixes bug labels Apr 4, 2023
@daavoo daavoo requested review from a team and dberenbaum April 4, 2023 17:21
"Adding it to git.",
path,
)
repo.scm.add([path])
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought it was simpler to be opinionated for this particular case instead of trying to handle arbitrary untracked files.

@codecov
Copy link

codecov bot commented Apr 4, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (cf52efb) 92.94% compared to head (0756205) 92.95%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9305   +/-   ##
=======================================
  Coverage   92.94%   92.95%           
=======================================
  Files         459      459           
  Lines       37112    37123   +11     
  Branches     5342     5344    +2     
=======================================
+ Hits        34493    34506   +13     
+ Misses       2089     2086    -3     
- Partials      530      531    +1     
Impacted Files Coverage Δ
tests/func/api/test_data.py 100.00% <ø> (+0.82%) ⬆️
dvc/api/data.py 87.14% <100.00%> (+2.42%) ⬆️
dvc/external_repo.py 83.10% <100.00%> (+0.11%) ⬆️
dvc/repo/experiments/run.py 97.91% <100.00%> (+0.29%) ⬆️
tests/func/experiments/test_queue.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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 in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@daavoo daavoo enabled auto-merge (rebase) April 5, 2023 07:57
Only if they are currently untracked and `--temp` or `--queue` is passed.

Closes #7842
@daavoo daavoo merged commit 3564813 into main Apr 5, 2023
@daavoo daavoo deleted the 7842-dvc-exp-run-queue-deleting-file-paramsyaml-for-each-command-call branch April 5, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: experiments Related to dvc exp bugfix fixes bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dvc exp run --queue: deleting file 'params.yaml' for each command call

4 participants