wmo-im / iwxxm

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

Issue with rule TAF.TAF-4 and TAF schema #104

Closed mgoberfield closed 6 years ago

mgoberfield commented 6 years ago

Under what circumstance would <iwxxm:previousReportAerodrome> be different from <iwxxm:aerodrome> ? In other words, if you are amending, correcting or canceling a TAF, aren't <iwxxm:aerodrome> and <iwxxm:previousReportAerodrome> always the same?

<iwxxm:previousReportAerodrome> seems to be a superfluous element and just adds unnecessary complexity and bloat to the resulting XML document.

blchoy commented 6 years ago

In IWXXM METAR/SPECI, TAF, AIRMET and SIGMET has attribute "status" to indicate the kind of message they are (e.g. amendment, cancellation, etc). Only TAF, AIRMET and SIGMET has elements making reference to previous messages. There is no use of the status attribute in advisories.

Seems to me that we may need to align the need of status indicator and references to previous messages. Furthermore, the relationship between "issueTime" and "previous time" under different situations (e.g. correction, amendment, cancellation, etc.) should be spelled out clearly as there is only one date-time group in TAC and it has different meanings under different situations.

I have no objection to remove "previousReportAerodrome" for the reason Mark mentioned.

blchoy commented 6 years ago

In the telecon on 24 Aug 2018, participants had no objection to remove "previousReportAerodrome".

blchoy commented 6 years ago

This is fixed in 3.0.0RC2.