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

update Earth discipline structure #70

Merged
merged 20 commits into from
Nov 20, 2023
Merged

update Earth discipline structure #70

merged 20 commits into from
Nov 20, 2023

Conversation

tomkralidis
Copy link
Collaborator

@tomkralidis tomkralidis commented Nov 15, 2023

Notes:

@tomkralidis tomkralidis added this to the INFCOM-3 milestone Nov 15, 2023
Copy link
Contributor

@antje-s antje-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

solution variant looks ok to me, even if I have seen a change in the standardisation of values from the solution variant with common topic-level value files over all domains

@tomkralidis
Copy link
Collaborator Author

TT-WISMD 2023-11-17:

  • removed earth-system-discipline-subcategory from topic-hierarchy.csv
  • add text to specification
    • domains SHALL use a hierarchy only (not relational)
    • experimental SHOULD NOT be a permanent solution

@tomkralidis
Copy link
Collaborator Author

@wmo-im/tt-wismd FYI this PR has now been updated per the discussion in #70 (comment). Please review and provide feedback or approve.

@tomkralidis tomkralidis merged commit 101caa3 into main Nov 20, 2023
1 check passed
@tomkralidis tomkralidis deleted the test-structure branch November 20, 2023 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add experimental level Organize level 8+ sub-topics provide publication artifacts
3 participants