search

wmo-im / iwxxm

XML schema and Schematron for aviation weather data exchange
https://old.wmo.int/wiswiki/tiki-index.php%3Fpage=TT-AvXML
48 stars 22 forks source link

Constrain IWXXM to 2D spatial CRSs Issue #140 #151

Closed mgoberfield closed 5 years ago

mgoberfield commented 5 years ago

140 Unable to obtain full list of allowed srsNames for aixm:ARP. Please review and update.

blchoy commented 5 years ago

Thanks @mgoberfield for your effort. In a separate discussion with @marqh, we come to the consensus that:

  1. It will be difficult to have/maintain a comprehensive list of all suitable CRS
  2. We have removed all RDF files that comes with iwxxm.sch but provided a script to download them directly from the Codes Registry. We believe this ensures freshness of the RDFs, but at the same time complicates the case for CRS as there is no single place to download the list unless we maintain our own

So we will only check that srsDimension should be equal to 2 when srsName is present, but not the value of srsName. I know this is weird but we are happy to discuss.

mgoberfield commented 5 years ago

Choy, Will make adjustments then, and correct the METAR which caused validation to fail.

On Tue, Apr 2, 2019 at 2:00 AM BL Choy notifications@github.com wrote:

Thanks Mark for your effort. In a separate discussion with Mark Hedley (how can we address people with the same first name without mentioning their family names?), we come to the consensus that:

  1. It will be difficult to have/maintain a comprehensive list of all suitable CRS
  2. We have removed all RDF files that comes with iwxxm.sch but provided a script to download them directly from the Codes Registry. We believe this ensures freshness of the RDFs, but at the same time complicates the case for CRS as there is no single place to download the list unless we maintain our own

So we will only check that srsDimension should be equal to 2 when srsName is present, but not the value of srsName. I know this is weird but we are happy to discuss.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/wmo-im/iwxxm/pull/151#issuecomment-478856247, or mute the thread https://github.com/notifications/unsubscribe-auth/AHfSTRLM3WmqDFpgg6HPC-wtt86JRaKaks5vcvHkgaJpZM4cWePV .

blchoy commented 5 years ago

We discussed. As there is no official list of CRS right now and it will be a hard work for WMO to maintain its own list of CRS we will not be able to take this approach at the moment. Otherwise this is a sensible way of mandating the use of 2D CRS.