Add progress bar for downloads about pooch HOT 3 CLOSED

Hi @danshapero thanks for opening the issue! It would be great to have tqdm support in our downloader. The change should be made to HTTPDownloader. Since this is not the current behavior, we should probably make it optional at first. If we want to make this the default, we can include a notice that it will be the default in a future release and a way to opt out so people can have time to adapt. Most projects use small test data so I imagine most people won't want this on by default.

One way to implement this would be to introduce a new progressbar=False argument for HTTPDownloader and add tqdm to the download loop if True. This way, the bar will be off by default and people can turn it on by passing downloader=HTTPDownloader(progressbar=True) to Pooch.fetch.

This is a good reference for how to implement this: https://stackoverflow.com/a/37573701/1259381

What do you think?

from pooch.

Suits me, I've used tqdm before. Do you want to make it a hard dependency (add to setup.py and requirements.txt) or an optional one?

from pooch.

It can be optional for now since the behavior is off by default.

from pooch.