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

Fix IWXXM 3.0 schematron rules for METAR/SPECI, AIRMET and SIGMET products. #317

Open
mgoberfield opened this issue Feb 21, 2023 · 2 comments
Assignees
Labels

Comments

@mgoberfield
Copy link
Contributor

mgoberfield commented Feb 21, 2023

Describe the bug
Two bugs were discovered with IWXXM 3.0 business rules which caused properly constructed IWXXM 3.0 XML METAR, SIGMET and AIRMET documents to fail schematron validation. This will cause consumers performing XML validations to erroneously flag, and worse, possibly reject the data.

Refer to issues #197 and #208 for more detailed write-ups on these two schematron bugs which were fixed in later releases of IWXXM.

To Reproduce
Steps to reproduce the behavior:
XML documents which trigger the false errors are attached, along with error messages from the OxygenXML application.

  1. Import the XML document into an application, such as OxygenXML, or against a command-line application such as CRUX that can perform schema and schematron validation.

  2. Be sure that the application or tool uses the IWXXM 3.0 schema and schematron files when performing schema and business-rules validation.

  3. For METARs, the schematron validation step erroneously flags every report in the as exceeding extension block content length (5000 characters). Clearly by visual inspection, each METAR report has much less than that.

  4. For LSCA31 SIGMET the timeIndicator is "OBSERVATION", and the validPeriod beginning time is later than the phenomenonTime consistent with schematron rules in later releases of IWXXM.

Expected behavior
All XML documents to pass IWXXM 3.0 schematron validation step. If you need more examples of SIGMETs and AIRMETs, I can provide them.

IWXXM 30 Schematron.zip

@blchoy blchoy self-assigned this May 31, 2023
@blchoy
Copy link
Member

blchoy commented May 31, 2023

Looking back these known bugs had been fixed in IWXXM 3.0-dev but this version has never been published after a debate. In order not to re-invent the wheel I have revived this branch and renamed it as v3.0.1RC1. I confirmed that both examples passed the schematron tests in iwxxm.sch. The remaining question is how to deal with this; are we going to leave it as a branch of the GitHub repository as unofficial materials? @amilan17 we need Secretariat's advise.

Note that even though I renamed the branch to v3.0.1RC1, the schema version is still indicated as 3.0-dev. If we decided to leave the branch there we will need to decide the version number the schemas should carry and make necessary adjustments.

@amilan17 amilan17 self-assigned this Jul 6, 2023
@amilan17
Copy link
Member

amilan17 commented Jul 6, 2023

https://github.com/wmo-im/iwxxm/wiki/TT-AvData-Teleconference-2023-Jul-6 notes:

@mgoberfield will test branch.

@amilan17 will work on publication options for the MoC vI.2, schematron and notifications needed.
The ONLY change is the schematron.

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

When branches are created from issues, their pull requests are automatically linked.

3 participants