-
Notifications
You must be signed in to change notification settings - Fork 46
Make CMOR tables configurable through new configuration system and deprecate config-developer.yml #2946
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
Merged
Merged
Make CMOR tables configurable through new configuration system and deprecate config-developer.yml #2946
Changes from 31 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
5ae3e93
Make CMOR tables configurable through new configuration system
bouweandela aaaeb9e
Fix another test
bouweandela ea4aa6a
Add docstrings and type hints
bouweandela 64f365b
Gracefully handle missing out_name in CMIP5-style CMOR tables and updβ¦
bouweandela 1b58631
Merge branch 'main' into new-config-cmor
bouweandela 7e74268
Merge branch 'main' of github.com:ESMValGroup/ESMValCore into new-conβ¦
bouweandela d676fbd
Remove config_developer_file from default configuration
bouweandela d0090d4
Use explicit configuration instead of YAML anchors
bouweandela d11aa25
Merge branch 'main' into new-config-cmor
valeriupredoi 33ce271
Small improvements
bouweandela ed35a66
Do not load CMOR tables from built-in config-developer.yml prior to lβ¦
bouweandela 4e11d10
Add note to docs about what to do if CMOR_TABLES is empty
bouweandela ab823bd
Add upgrade instructions for users moving away from config-developer.yml
bouweandela d08827f
Fix no CMOR table case and update more docs
bouweandela 01129d4
Fix documentation build
bouweandela f80b1a4
Further docs improvements
bouweandela d14d09f
Add test
bouweandela e2dfa5d
Improve test coverage
bouweandela 74f27cc
Test failure to load cmor tables cases
bouweandela 0cba683
More tests
bouweandela adff349
Add another test and update to pytest style tests
bouweandela 067c6f0
Add more tests
bouweandela 565849e
Merge branch 'main' of github.com:ESMValGroup/ESMValCore into new-conβ¦
bouweandela 69487f0
Add more tests
bouweandela 8c13364
Add copilot instructions
bouweandela dd2f560
Implement suggestions from code review
bouweandela 08924a9
Address more review comments
bouweandela 7a6c11a
Fix typo in link
bouweandela aff84c8
Deprecate esmvalcore.cmor.table.CMOR_TABLES becuase of #2954
bouweandela 4199ca5
Small improvements
bouweandela fda2000
Improve AI instructions
bouweandela eb36404
Merge branch 'main' into new-config-cmor
bouweandela bf8d6e5
Merge branch 'main' into new-config-cmor
valeriupredoi 5519c3f
Improve type hint
bouweandela a18ee08
Add punctuation
bouweandela 2b711c9
Clarify data requirements for non-cmorized data
bouweandela e0edb85
Spelling
bouweandela e502d7c
Merge branch 'main' into new-config-cmor
valeriupredoi daec426
Add examples to docstrings
bouweandela fea3efd
Add link for branded variable background
bouweandela 403bd62
Fix docs
bouweandela 739fe73
Better error messages
bouweandela 5a1852a
Add tests
bouweandela 8c0cbbd
Separate custom tables and reduce race conditions in tests
bouweandela ef94629
Merge branch 'main' into new-config-cmor
bouweandela b3de2a8
Restore custom tables test from config-developer and add another patcβ¦
bouweandela e67aea3
Add another deprecation warning
bouweandela 1dec95f
Ensure config_developer_file populates esmvalcore.cmor.table.CMOR_TABβ¦
bouweandela 610c23d
Fix support for legacy data sources without custom config-developer.yml
bouweandela b5064fd
Improve test coverage and fix bug
bouweandela eb3306c
Merge branch 'main' of github.com:ESMValGroup/ESMValCore into new-conβ¦
bouweandela 81317b1
Update siextent units in CMIP6 custom table
bouweandela a982de2
Improve docs with suggestions by @schlunma
bouweandela 5a2147d
Remove impractical suggestion on how to organize data
bouweandela 2ee9aad
Make defaults/cmor_tables.yml a link
bouweandela b43a466
Add note about which files are read
bouweandela File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| doc/contributing.rst |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.