astropy / astropy/pyvo

ENH: Handling open RANGE intervals for POS options: handling Inf

Open
#598 5 comments 0 reactions 0 assignees View on GitHub
component: SIA
Dominant language
Python
Stars
88
Forks
64
Avg merge
38m
Merged PRs (30d)
1

Description

I was following up on recent reports of SIAv2 inconsistencies, and run into the issues that currently we don't allow `+/- Inf` for POS values, though that standard (e.g. SIA2 2.1.1 section) lists them as valid values for e.g around the poles or for all sky.

Should we allow these? It will need a bit of a hackery in the validator, but can make it work.
```
# North pole: (0, 360, 89, +Inf)]
# All sky: (-Inf +Inf -Inf +Inf)
```

(Also, with the very recent #594 it became clear that we have these position parsers both in DAL and in registry. Now the two are not exactly the same, so may not worth spending too much effort to cleanup, but still, we may consider doing it at some point.

Contributor guide

No contributing guide indexed for this repository

Research direction

Read the POS validators and position parsers in the DAL and registry areas, then compare their current handling of finite values with SIA2 section 2.1.1. Use the North Pole and all-sky examples in the issue as cases; done means valid open ranges containing +/-Inf are accepted consistently, with the parser differences understood or addressed.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.