download_dataset has a quiet argument to suppress rclone's stdout (and other stuff). In #48 we added progress to upload_dataset (which decides whether to pass --progress to rlcone). We should make the arguments consistent somehow, but their behavior is subtly different.
Just use quiet, deprecate progress.
Just use progress, deprecate quiet.
Use both, because their behavior is different.
I'm partial to (1): rclone is an implementation detail, so the only thing we can really promise is that there is no output at all, from anywhere.
download_datasethas aquietargument to suppressrclone's stdout (and other stuff). In #48 we addedprogresstoupload_dataset(which decides whether to pass--progresstorlcone). We should make the arguments consistent somehow, but their behavior is subtly different.quiet, deprecateprogress.progress, deprecatequiet.I'm partial to (1):
rcloneis an implementation detail, so the only thing we can really promise is that there is no output at all, from anywhere.