roboflow-python icon indicating copy to clipboard operation
roboflow-python copied to clipboard

.download() re-downloads the same version even if it already exists on disk

Open yeldarby opened this issue 2 years ago • 5 comments
trafficstars

The default behavior should be to use the local copy since versions are frozen in time. Perhaps we could provide a force=true option that deletes the existing one first to clear local changes.

But especially for large datasets this is wasting time and bandwidth when re-training.

Example from our YOLOv8 notebook:

image

yeldarby avatar Feb 01 '23 14:02 yeldarby

Hi @yeldarby :wave: , thanks for reporting it. I'll have a look :)

@yeldarby note that the .download , accepts a parameteroverwrite that is set to True by default. Thus, everytime you downlod a dataset, it will overwrite your local one.

You can remove this behaviour by setting overwrite = False. Let me know if this helps

Should we make the default be False? When would you want to overwrite it? (Exported versions are immutable)

yeldarby avatar Mar 02 '23 18:03 yeldarby

@yeldarby I am in support of this change. Want me to file a PR?

capjamesg avatar Sep 01 '23 11:09 capjamesg

@yeldarby Please see https://github.com/roboflow/roboflow-python/pull/213.

capjamesg avatar Dec 28 '23 10:12 capjamesg