FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ESA JWST Migration to PyVO by jespinosaar · Pull Request #3592 · astropy/astroquery · GitHub

/ astroquery Public

ESA JWST Migration to PyVO - #3592

Open
jespinosaar wants to merge 18 commits into
astropy:mainfrom
esdc-esac-esa-int:ESA_jwst-pyvo_migration
ConversationCommits (18)ChecksFiles changed
Open

ESA JWST Migration to PyVO#3592
jespinosaar wants to merge 18 commits into
astropy:mainfrom
esdc-esac-esa-int:ESA_jwst-pyvo_migration

Conversation

jespinosaar commented May 14, 2026

Copy link
Copy Markdown
Contributor

Dear Astroquery team,

This PR is an upgrade of the ESA JWST module, replacing the previous TAP implementation with the newest one based on PyVO, already applied to other Archives.

With this, code has been cleaned and classes are now making use of the EsaTap class. Together with this, now we have added the possibility of uploading tables.

Thanks to @lmasselo for doing this! Please add her to the list of contributors as well (we currently have a lot of people working on Astroquery).

Please let us know if you have any comment.

Kind regards,
jespinosa

jespinosaar added this to the 0.4.12 milestone May 14, 2026
jespinosaar self-assigned this May 14, 2026

codecov Bot commented May 14, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.61165% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.20%. Comparing base (a0ec925) to head (3c81d43).
⚠️ Report is 200 commits behind head on main.

Files with missing lines Patch % Lines
astroquery/esa/jwst/core.py 80.88% 26 Missing ⚠️
astroquery/esa/utils/utils.py 64.44% 16 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3592      +/-   ##
==========================================
- Coverage   73.23%   73.20%   -0.03%     
==========================================
  Files         226      225       -1     
  Lines       21010    21055      +45     
==========================================
+ Hits        15386    15413      +27     
- Misses       5624     5642      +18

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

jespinosaar commented May 14, 2026

Copy link
Copy Markdown
Contributor Author

We will work next week on increasing the coverage. In the meantime, we can start having a look at this PR, many thanks!

jespinosaar commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

It seems all the checks are passing now, except for the oldest version for all dependencies. This seems to be an unrelated error.

Please let me know if you have any comment. Thanks in advance!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

jespinosaar

Labels

esa.jwst

Projects

None yet

Milestone

0.4.12

Development

Successfully merging this pull request may close these issues.

1 participant

Back | FazBrowse Home | New Git URL