-
Notifications
You must be signed in to change notification settings - Fork 24
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
Eliminate Gen-Ens-Prod warning when parsing the nbhrd_prob dictionary #2224
Labels
component: code cleanup
Code cleanup and maintenance issue
priority: medium
Medium Priority
requestor: DTC/Ensembles
DTC Ensembles Team
type: enhancement
Improve something that it is currently doing
Milestone
Comments
JohnHalleyGotway
added
type: enhancement
Improve something that it is currently doing
component: code cleanup
Code cleanup and maintenance issue
priority: medium
Medium Priority
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
requestor: METplus Team
METplus Development Team
labels
Aug 4, 2022
JohnHalleyGotway
added
requestor: DTC/Ensembles
DTC Ensembles Team
and removed
requestor: METplus Team
METplus Development Team
labels
Aug 4, 2022
JohnHalleyGotway
changed the title
Eliminate the warning message when parsing the nbhrd_prob dictionary from Ensemble-Stat and Gen-Ens-Prod
Eliminate the warning message when parsing the nbhrd_prob dictionary from Gen-Ens-Prod
Nov 2, 2022
JohnHalleyGotway
changed the title
Eliminate the warning message when parsing the nbhrd_prob dictionary from Gen-Ens-Prod
Eliminate Gen-Ens-Prod warning when parsing the nbhrd_prob dictionary
Nov 2, 2022
JohnHalleyGotway
added a commit
that referenced
this issue
Nov 2, 2022
…ollowing types of warnings: WARNING: process_command_line() -> control_id is set in the config file but control file is not provided with -ctrl argument
JohnHalleyGotway
added a commit
that referenced
this issue
Nov 2, 2022
…g the field entry for the nbrhd_prob. This seems preferable to all the duplication involved in writing a new parse_conf_nbrhd_prob() function.
15 tasks
JohnHalleyGotway
moved this from Todo
to Pull Request Review
in MET-11.0.0-beta5 (11/20/22)
Nov 2, 2022
JohnHalleyGotway
added a commit
that referenced
this issue
Nov 3, 2022
JohnHalleyGotway
added a commit
that referenced
this issue
Nov 3, 2022
Repository owner
moved this from Pull Request Review
to Done
in MET-11.0.0-beta5 (11/20/22)
Nov 3, 2022
JohnHalleyGotway
removed
the
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
label
Oct 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
component: code cleanup
Code cleanup and maintenance issue
priority: medium
Medium Priority
requestor: DTC/Ensembles
DTC Ensembles Team
type: enhancement
Improve something that it is currently doing
Describe the Enhancement
This issue was raised by the DTC Ensembles team. When running Ensemble-Stat via METplus, the following WARNING message appeared:
I traced through the logic and found the following:
But "field" is NOT included in the default nbrhd_prob dictionary, so that entry will not in general be found.
However the config file logic continues searching higher and higher levels of context. When "field" is found at one of those higher levels of context this warning message is triggered.
For example, adding the following to the Ensemble-Stat config file triggers the warning:
Recommend writing/using a specific parse_conf_nbrhd_prob() function instead of reusing parse_conf_nbrhd().
Time Estimate
2 hours.
Sub-Issues
Consider breaking the enhancement down into sub-issues.
None needed.
Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
No impacts.
Enhancement Checklist
See the METplus Workflow for details.
Branch name:
feature_<Issue Number>_<Description>
Pull request:
feature <Issue Number> <Description>
Select: Reviewer(s) and Linked issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
The text was updated successfully, but these errors were encountered: