Skip to content

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Nov 26, 2022

Easy fix, but no one picked it up in the past year. The PR also works towards #1746
This should close #2190

@bsipocz bsipocz added this to the v0.4.7 milestone Nov 26, 2022
@codecov
Copy link

codecov bot commented Nov 26, 2022

Codecov Report

Merging #2601 (ed9cdb7) into main (0eb8daa) will increase coverage by 0.12%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #2601      +/-   ##
==========================================
+ Coverage   64.08%   64.21%   +0.12%     
==========================================
  Files         131      130       -1     
  Lines       16927    16885      -42     
==========================================
- Hits        10847    10842       -5     
+ Misses       6080     6043      -37     
Impacted Files Coverage Δ
astroquery/skyview/core.py 90.29% <50.00%> (ø)
astroquery/esa/hubble/core.py 83.63% <0.00%> (-2.84%) ⬇️
astroquery/utils/commons.py 75.14% <0.00%> (-1.58%) ⬇️
...roquery/ipac/nexsci/nasa_exoplanet_archive/core.py 57.08% <0.00%> (-0.47%) ⬇️
astroquery/esa/jwst/core.py 75.43% <0.00%> (ø)
astroquery/utils/download_file_list.py
astroquery/utils/tap/core.py 44.55% <0.00%> (+0.23%) ⬆️
astroquery/jplhorizons/core.py 65.62% <0.00%> (+0.32%) ⬆️
astroquery/mast/auth.py 60.86% <0.00%> (+0.45%) ⬆️
astroquery/utils/cleanup_downloads.py 93.33% <0.00%> (+26.66%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bsipocz bsipocz mentioned this pull request Nov 26, 2022
62 tasks
@bsipocz bsipocz merged commit f9b659d into astropy:main Nov 27, 2022
@bsipocz bsipocz deleted the BUG_skyview_radius branch April 9, 2024 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The radius in SkyView get_image_list is actually diameter
2 participants