Skip to content
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

input PARAMETERS with limited list of values : better description #1

Open
Bonnarel opened this issue May 7, 2020 · 1 comment
Open
Labels

Comments

@Bonnarel
Copy link
Collaborator

Bonnarel commented May 7, 2020

Several SIAP2.0 parameters have a limited list of possible values
– Some have lists limited by protocol (and obscore)

  • POL (Stokes, LINEAR, etc..)
  • DPTYPE (image, cube, visibility, timeseries ;..)
  • CALIB : levels
  • FORMAT : fts, jpeg , png, etc..

– Some have free string values

  • COLLECTION (HST, WISE, etc...),
  • FACILITY (VLT, Keck, Chandra),
  • INSTRUMENT (ACS, MEGACAM, etc.)

– PARAMETERs less useful if we have no prior idea of their possible
values.
It often happens that services do not provide these lists of parameters although section 2.1.1
However Section 2.1.20 of SIAP2.0 states :
Service PARAMETER self description
Any service may include a DataLink service descriptor in the vOTable output to describe itself. this descriptor would describe the supported query parameters (standard and custom) including list of values for those with a fixed list (eg. COLLECTION INSTRUMENT FACILITY DPTYPE CALIB and FORMAT) *

This will allow to discover "possibilities" of the service prior to usage and optimize the queries

The "MAY" is obviously too weak. Version 2.1 could state SHOULD or MUST

@pdowler
Copy link
Collaborator

pdowler commented Apr 26, 2022

I'm not sure changing from may to should would really accomplish anything.

Changing this to MUST isn't compat enough for SIA-2.1, but it could pass muster for SDA-2.1 (#10)... let's see what Markus says about when to use MUST in ops.

@pdowler pdowler added the see DAP label Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants