-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
exp save: update --help #8729
exp save: update --help #8729
Conversation
@@ -68,7 +68,7 @@ def add_parser(experiments_subparsers, parent_parser): | |||
"-f", | |||
"--force", | |||
action="store_true", | |||
help="Replace local experiment already exists.", | |||
help="Replace local experiment if it already exists.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could also be plural?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(same for exp push
)
Codecov ReportBase: 93.50% // Head: 93.50% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #8729 +/- ##
=======================================
Coverage 93.50% 93.50%
=======================================
Files 457 457
Lines 36203 36203
Branches 5244 5244
=======================================
Hits 33850 33850
Misses 1845 1845
Partials 508 508
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. |
@@ -49,7 +49,7 @@ def add_parser(experiments_subparsers, parent_parser): | |||
"--force", | |||
action="store_true", | |||
default=False, | |||
help="Save even if hash value for dependencies/outputs changed.", | |||
help="Replace experiment if it already exists", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
help="Replace experiment if it already exists", | |
help="Replace experiment if it already exists.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about mentioning that "save also errors out if nothing changed in the workspace, --force overrides this" (from iterative/dvc.org#4200 (comment))?
We can have that in the docs only if you want to keep this short.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer to keep short, Just in the docs is fine I guess.
Note that I updated the docs around this in iterative/dvc.org@0a08f16 (PTAL). |
912b1d0
to
4b441ad
Compare
4b441ad
to
0f8f495
Compare
* start: `exp save`-related clarifications Per #4166 (review) * guide: `exp save`-related updates Per #4166 (review) * ref: `exp save`-related improvements Per #4166 (review) * ref: copy edits * ref: improve value prop for `exp save/run` Per #4200 (review) * ref: update `exp push/pull -f` Per iterative/dvc#8729 * ref: update `exp save -f` Per #4200 (review) * Update content/docs/command-reference/exp/run.md * Update content/docs/command-reference/exp/save.md * Update content/docs/user-guide/experiment-management/experiments-overview.md * Restyled by prettier (#4213) Co-authored-by: Restyled.io <commits@restyled.io> * Update content/docs/command-reference/exp/pull.md * Update content/docs/command-reference/exp/push.md * Update content/docs/command-reference/exp/save.md * Update content/docs/start/experiment-management/experiments.md * Restyled by prettier (#4337) Co-authored-by: Restyled.io <commits@restyled.io> --------- Co-authored-by: daniele <36171005+dtrifiro@users.noreply.github.com> Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Dave Berenbaum <dave@iterative.ai>
exp save --force
exp pull --force