Skip to content

Support nested variable entries in CMOR tables#1

Draft
Copilot wants to merge 3 commits into
mainfrom
copilot/open-pr-to-main-holding-work
Draft

Support nested variable entries in CMOR tables#1
Copilot wants to merge 3 commits into
mainfrom
copilot/open-pr-to-main-holding-work

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 17, 2026

Summary

  • add backward-compatible support for nested variable_entry objects that group brands under each variable name
  • keep existing flat table loading and variable lookup behavior unchanged by reconstructing the legacy variable id internally
  • add CMIP6 and CMIP7 regression coverage for nested branded and unbranded entries

Validation

  • LDFLAGS='-lm' python -m pip install --no-deps --no-build-isolation .
  • python Test/test_python_branded_variable.py
  • python Test/test_cmor_CMIP7.py

Notes

  • parallel_validation passed CodeQL and only left minor maintainability suggestions in the tests/loader

Copilot AI and others added 3 commits April 17, 2026 22:26
Agent-Logs-Url: https://github.com/ilaflott/cmor/sessions/789e2979-03f5-480c-a8d7-6487d07b9c72

Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ilaflott/cmor/sessions/789e2979-03f5-480c-a8d7-6487d07b9c72

Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
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.

2 participants