astropy / astropy/pyvo

MIVOT: make the creation of SkyCoord instances bullet-proof

Open
#680 0 comments 2 reactions 0 assignees View on GitHub
bug component: mivot enhancement
Dominant language
Python
Stars
88
Forks
64
Avg merge
38m
Merged PRs (30d)
1

Description

While testing the `mivot.features` on Simbad query response, I've seen that `SkyCoordBuilder`often fails due to the lack of input data, or even worse due to their inconsistency.
Simbad is a very good test bed because the data aggregated in query response are sparse.

- The missing/wrong parameters must be checked individually and possibly flagged as ignored
- Errors risen by Astropy when the instantiation fails, must be properly caught.
- A specific test module must be setup for this purpose.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating SkyCoordBuilder and the MIVOT handling used for Simbad query responses. Add a dedicated test module covering missing or inconsistent parameters, ignored values, and Astropy instantiation failures; done means these cases are handled without uncaught errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 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.