-
Notifications
You must be signed in to change notification settings - Fork 394
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
add repro --recursive in Repro command reference #396
Changes from 12 commits
13779e3
0d71a0e
bbc2858
fecfa43
30e1a12
f4b9d40
5ce14e5
ba267ed
d248f79
8b568df
1ee43e9
3f7120b
3457ad8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,7 @@ Remove unused objects from cache or remote storage. | |
|
||
```usage | ||
usage: dvc gc [-h] [-q | -v] [-a] [-T] [-c] | ||
[-r REMOTE] [-f] [-j JOBS] | ||
[-r REMOTE] [-f] [-j JOBS] [- R] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There's a space between
Which doesn't include There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ok and it dvc -R is not mentioned in docs for a no of other commands as well like There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks. So did you update this usage and remove the Could you please open an issue for the docs that are missing the -R option and need update in their usage blocks? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Opened issue iterative/dvc#2154_ There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. there still a space in this file between - and R There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @kurianbenoy Where did you get |
||
[-p [PROJECTS [PROJECTS ...]]] | ||
``` | ||
|
||
|
@@ -47,6 +47,11 @@ usually helps to save some space. | |
- `-r`, `--remote` - name of the remote storage to collect unused objects from | ||
if `-c` option is specified. | ||
|
||
- `-R`, `--recursive` - It removes the unused objects from cache or remote | ||
storage and determines the DVC-files to download by searching the named | ||
directory/targets for DVC-files to download data for. `targets` values is | ||
expected to be a directory path and also files as well. | ||
|
||
- `-j`, `--jobs` - garbage collector parallelism level. The default value is | ||
`4 * cpu_count()`. For SSH remotes default is 4. For now only some phases of | ||
GC are parallel. | ||
|
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.
commit cache-
?This comment was marked as off-topic.
Sorry, something went wrong.
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.
Wait... Me again, SORRY.
Actually I DID update this option already. (I had forgotten to push the commit to my PR until now). See for example https://github.com/iterative/dvc.org/pull/441/files#diff-cf1201ba4e234c56d863d0adad07fc7aR82
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.
@jorgeorpinel have you also added -R to repro? That was the original scope of this issue.
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.
seems like you have not added -R option in
dvc commit
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 had not until now. Added the change o my newest PR now (See #[email protected])