-
Notifications
You must be signed in to change notification settings - Fork 1
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
Marine templates #54
Marine templates #54
Conversation
Update of synop and temp to reference codes.wmo.int for Earth system discipline rather than github.
@david-i-berry could we have used the MCF format here for the template? This would ensure / increase compatibility between various workflows in wis2box. |
@david-i-berry I have deployed this on the test-server, see https://wis2box-test.wis2dev.io/wis2box-webapp/dataset_editor, the label for ship-hourly-template is incorrect leading to duplicate labels in the dropdown: |
The dropdown is fixed, I created 2 new datasets using the templates: @tomkralidis can you please review the result is as intended: |
@maaikelimper can you the links as published via MinIO (which is what is sent to WIS2 proper)? I am also seeing that, in both examples, "themes": [
{
"concepts": [
{
"id": "weather",
"title": "Weather"
}
],
"scheme": "http://codes.wmo.int/wis/topic-hierarchy/earth-system-discipline"
}
], |
Here are the public href links are requested: Please note that the problem you reported, about the scheme needing to be a string not an array, is unrelated to this PR, the same can be seen for other datasets using the existing synop template: |
Thanks. I've opened wmo-im/wis2box#728 to track the WCMP2 encoding fix. |
Addition of WCMP2 metadata templates for drifters and ship based surface weather observations.