opengeospatial / ogcapi-connected-systems

Public Repository for the Connected Systems SWG
Other
7 stars 6 forks source link

Options for specifying the position of a physical process #39

Closed autermann closed 4 weeks ago

autermann commented 5 months ago

The SensorML 2.1 XML encoding allows the position of a physical process to be one of the following:

While the JSON schemas only allows GeoJSON Point, GeoPose and swe:Vector.

Is this intended? Should be trivial to include the other definitions in the oneOf.

alexrobin commented 5 months ago

It was kind of intended :-) When I wrote the JSON schemas, I thought maybe we should simplify this part, but to be honest I was unsure about this. So I'm glad you brought this up.

I think the Text option can still be useful.

Certainly most cases of DataRecord usages would now be handled by the new GeoPose / RelativePose options introduced in JSON.

And my thinking was that now that we live in an API world, pointing to an AbstractProcess or DataArray does not make as much sense, and I would rather reference the datastream that provides the position data (whether it is provided by a process or a GPS sensor doesn't matter).

I'm open to suggestions though. Do you have use cases where you use the advanced options? Do you think we should try to stay 100% compatible with the XML encodings?

I don't have a strong opinion on this.

alexrobin commented 5 months ago

I think we could also keep all options and mark some as deprecated in the JSON encodings. And deprecated would mean implementations are not required to implement them.

alexrobin commented 4 months ago

Discussed during 02/15 Telecon. We will add back support for all SensorML 2.1 options into the JSON schema to maintain full compatibility

alexrobin commented 4 months ago

See 7e46046dbc1aa39e268b6c9981b4a8b453b4d4cc

alexrobin commented 4 weeks ago

Reopening to make sure I don't forget

alexrobin commented 4 weeks ago

Nevermind it's done