astropy / astropy/astroquery

Progressbar for ESO get_header

Open
#3,000 5 comments 0 reactions 0 assignees View on GitHub
enhancement eso infrastructure
Dominant language
Python
Stars
791
Forks
451
Avg merge
1d 3h
Merged PRs (30d)
4

Description

I am working on a package that requires downloading a lot of headers for files from the ESO archive, which can take quite some time. I was wondering what the astroquery stance on progressbar utils is? Getting a progressbar would be very easy for this function by simply wrapping the for-loop in astroquery.eso.core.get_headers with the tqdm package and adding an additional keyword, e.g. "progress" to the function to toggle this (that's what I do in my local install). The only code line that needs to be changed is 572:
`for dp_id in product_ids` -> `for dp_id in tqdm(product_ids)`

However, this requires adding tqdm as a dependency. I'm sure there are many parts of astroquery that would benefit from having a simple progressbar, but I don't know what astroquery's policy for adding dependencies is.

Contributor guide

Open the contributing guide

Research direction

Start in astroquery.eso.core.get_headers at the loop around line 572, and review how product_ids are downloaded. Check the project's dependency policy and determine whether a progress option using tqdm is acceptable; done means the proposed behavior and dependency choice are settled and documented in the implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.