dtcenter / METplus

Python scripting infrastructure for MET tools.
https://metplus.readthedocs.io
Apache License 2.0
97 stars 37 forks source link

Enhancement: MvMode config format changes #2429

Closed davidalbo closed 3 months ago

davidalbo commented 9 months ago

Describe the Enhancement

Based on this issue, https://github.com/dtcenter/MET/issues/2706, for Multivariate Mode, took out the intensity_flag parameter and added two new integer array parameters in it's place:

multivar_intensity_compare_fcst = [ ];
multivar_intensity_compare_obs = [ ];

so METplus environment variables will need some tweaking. Everything else is the same.

Example with 3 input forecasts and obs. Previous: intensity_flag=[FALSE, FALSE, FALSE]

New:

multivar_intensity_compare_fcst = [ ];
multivar_intensity_compare_obs = [ ];

Previous: intensity_flag=[FALSE, TRUE, TRUE]

New:

multivar_intensity_compare_fcst = [2, 3 ];
multivar_intensity_compare_obs = [2, 3];

Time Estimate

1 day

Sub-Issues

Consider breaking the new feature down into sub-issues.

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.

Enhancement Checklist

See the METplus Workflow for details.

georgemccabe commented 9 months ago

Hi @davidalbo, thank you for creating this issue. This is more of an enhancement than a new use case, so I modified the template to use the enhancement issue sections and checklist. The sections you had questions about are not found in the enhancement template, so you don't need to worry about them.

hertneky commented 9 months ago

@davidalbo Just noting that with these changes, the use-case would also need to be updated, which would be a minor change.