-
Notifications
You must be signed in to change notification settings - Fork 2
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
Missing GeoJson geometries #74
Comments
@outsideMyBox the WNM only allows Point and Polygon and does not allow for other types of GeoJSON geometries. |
@outsideMyBox the thinking about constraining geometry to Point or Polygon was to keep the profile simple based on the most common geometries for use cases in the context of the notification message itself (which is designed to provide a high level overview/simple filtering). The discovery metadata (and data, of course) can have any geometric complexity as needed. |
@tomkralidis this question is coming from my team because we commonly have products with two swaths (so two polygons). See for instance this leo product. It is a very common scenario to have users only interested in a region of the word while our LEO satellites are scanning the entire globe. Users would then like to use the notification messages to filter not only in time by in also in space. My proposal would be to allow complex geometries in the WNM standard (they validate) but recommend in the guide to use simple ones as it will be the most common case. What do you think ? |
TT-WISMD 2024-11-27
It would be valuable to assess this requirement in the next 12-18 months of WIS2 operations (i.e. how WIS2 data consumers are working with WNM as clients) |
Hi,
in notificationMessageGeoJSON.yaml only two geometry types are included: point and polygon.
But GeoJson supports the following geometry types: Point, LineString, Polygon, MultiPoint, MultiLineString, and MultiPolygon. Defined also there.
Is it intentional to restrict to these 2 types or should the other ones be included as well?
Thanks!
The text was updated successfully, but these errors were encountered: