-
Notifications
You must be signed in to change notification settings - Fork 22
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
The codes.wmo.int-49-2-AerodromeRecentWeather.rdf files in several IWXXM versions published at https://schemas.wmo.int/iwxxm/ do not contain RERASN #328
Comments
a possible solution: |
Checking the file codes.wmo.int-49-2-AerodromeRecentWeather.rdf in various locations, including schemas.wmo.int as well as my own SVN:
So the likely scenario is that:
Now the CI script codeListsToSchematron.py is intended to download RDFs from codes.wmo.int if they are not there. However, if an RDF does exist the download will be skipped. It seems to me that downloading should only be skipped if the ones on the repository is the same as those on codes.wmo.int. A separate issue will be raised to deal with the refinement of codeListsToSchematron.py. I agree with @amilan17's suggestion to (1) update codes.wmo.int-49-2-AerodromeRecentWeather.rdf for IWXXM versions 2.1.1, 2021-2 and 2023-1 on schemas.wmo.int (note that if there are zip files they should also be updated), and (2) put a note in the respective releases on GitHub saying that this file has been updated and if one downloaded the releases they should refer to schemas.wmo.int for the correct version of codes.wmo.int-49-2-AerodromeRecentWeather.rdf. |
Describe the bug
The codes.wmo.int-49-2-AerodromeRecentWeather.rdf files in several IWXXM versions published at https://schemas.wmo.int/iwxxm/ do not contain RERASN (https://codes.wmo.int/49-2/AerodromeRecentWeather/_RERASN that is mapped to http://codes.wmo.int/306/4678/RASN). RERASN was added to the code registry in #74 in IWXXM 2.1.1.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
All mentioned RDF files should contain
Screenshots
Desktop (please complete the following information):
Additional context
The text was updated successfully, but these errors were encountered: