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.

Sign up for GitHub

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

Jump to bottom

Want to add warning if BCVs are not included #32

Open
CMIP-IPO opened this issue Nov 28, 2024 · 1 comment
Open

Want to add warning if BCVs are not included #32

CMIP-IPO opened this issue Nov 28, 2024 · 1 comment
Assignees
@JamesAnstey @rigoudyg
Labels
High priority Issues which need to be addressed at highest priority

Comments

@CMIP-IPO
Copy link

No description provided.

@JamesAnstey JamesAnstey added the High priority Issues which need to be addressed at highest priority label Nov 28, 2024
@JamesAnstey
Copy link

Setting of priority levels in dreq_query.get_requested_variables ensures Core level (i.e., BCVs) are always included, i.e. it is not possible for the user to exclude the Core priority level.

In this PR, to be extra sure, added flag check_core_variables to this function (default: True) to check at the end just to be extra sure. This should resolve this issue for export_dreq_lists_json.py and workflow_example.py.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JamesAnstey JamesAnstey

@rigoudyg rigoudyg

Labels
High priority Issues which need to be addressed at highest priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JamesAnstey @rigoudyg @CMIP-IPO