Skip to content

[SIMBAD] add async_job option in all queries #3305

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 5, 2025

Conversation

ManonMarchand
Copy link
Member

This should fix #3205

It adds the possibility to launch SIMBAD queries in asynchronous mode. This mode allows to take timeout in account (this translates into executionDuration in the VOSI async standard).

Copy link

codecov bot commented Apr 28, 2025

Codecov Report

Attention: Patch coverage is 91.42857% with 3 lines in your changes missing coverage. Please review.

Project coverage is 69.88%. Comparing base (868c135) to head (50d397a).
Report is 31 commits behind head on main.

Files with missing lines Patch % Lines
astroquery/simbad/core.py 91.42% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3305      +/-   ##
==========================================
+ Coverage   69.82%   69.88%   +0.05%     
==========================================
  Files         232      232              
  Lines       19711    19757      +46     
==========================================
+ Hits        13764    13807      +43     
- Misses       5947     5950       +3     

☔ View full report in Codecov by Sentry.
📢 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.

Copy link
Contributor

@keflavich keflavich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, I have side conversation questions.

@bsipocz bsipocz added this to the v0.4.11 milestone Apr 28, 2025
Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks all good. Thanks!

Copy link
Member

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ManonMarchand.

Failures are unrelated and have been fixed in main already.

@bsipocz bsipocz merged commit 1379a6f into astropy:main May 5, 2025
3 of 10 checks passed
@ManonMarchand ManonMarchand deleted the simbad-async-sync branch May 7, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add possibility to chose between async/sync queries in SIMBAD methods
3 participants