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

Unions of interval, circle, etc #39

Open
mbtaylor opened this issue May 21, 2024 · 0 comments
Open

Unions of interval, circle, etc #39

mbtaylor opened this issue May 21, 2024 · 0 comments

Comments

@mbtaylor
Copy link
Member

At the DAL I session in Sydney today, @pdowler suggested that the intention of the Interval type in DALI is to support multiple intervals, and possibly similarly for other shape-like types such as Circle; this would address part of ESO's region declaration requirements that are currently provided using STC-S UNIONs.

As currently written the Intervals section 3.4 says:

Floating point interval values serialised in VOTable or service parameters must have the following metadata in the FIELD element: datatype="double" or datatype="float", arraysize="2", xtype="interval".

i.e. there is no reference to the possibility of storing multiple intervals in a single xtype="interval" field.

If the intention is to interpret an xtype="interval" field with 2 n elements as the union of n intervals, this should be stated explicitly. Similarly for other types where this makes sense: circle, range, point?

Then if interval is supposed to support multi-interval, circle multi-circle etc, I don't see why we need multipolygon alongside polygon. Although the array size is not fixed for polygon, the (NaN,NaN) delimiters mean that you can determine from looking at the content whether it's a single- or multi-polygon. The same could apply to shape.

Personally I think it would be fine to allow all of these to support unions of multiple instances of the named type. But there may be arguments against it that I haven't thought of.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant