-
Notifications
You must be signed in to change notification settings - Fork 26
Description
Describe the New Feature
There are many ways that plot_point_obs could be improved. Listed below are some recommended changes based on a recent met_help question from Binyu:
We should change plot_point_obs’s command line argument from:
-data_file path to gridded data file
To:
-plot_grid filename, grid name, grid spec
And then just reuse the same logic for parsing “to_grid” in regrid_data_plane. Still quietly support -data_file.
From binyu’s met_help question. She also want to filter obs by their height or pressure.
We could also add an optional plot_point_obs config file consisting of a “plot_options” array where each element is a dictionary defining a filter and set of plotting options. The logic would be... for each plot_options entry, loop through all of the input point observations and plot the matching dots. The alternative logic would be to only read the obs once, and then plot a dot using the matching plot_options entry. Or if none match, plot nothing. I think the second logic is simpler.
// Default config file
outline_color = “”;
fill_color = “red”;
fill_color_table = “”; // takes precedence over fill_color. Does this mean we need a color bar?
dotsize = 10; // could be a function of obs value?
message_type = [];
obs_var = [];
height_thresh = NA;
pressure_thresh = NA;
obs_thresh = NA;
obs_qc = [];
// what about convert and censor_thresh/censor_val?
plot_options = [
{ fill_color = “red”; }
];
Acceptance Testing
List input data types and sources.
Describe tests required for new functionality.
Time Estimate
3 days.
Sub-Issues
Consider breaking the new feature down into sub-issues.
- Add a checkbox for each sub-issue here.
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
- Select engineer(s) or no engineer required
- Select scientist(s) or no scientist required
Labels
- Select component(s)
- Select priority
- Select requestor(s)
Projects and Milestone
- Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
- Select milestone to next major version milestone or "Future Versions"
Define Related Issue(s)
Consider the impact to the other METplus components.
New Feature Checklist
See the METplus Workflow for details.
- Complete the issue definition above, including the Time Estimate and Funding source.
- Fork this repository or create a branch of develop.
Branch name:feature_<Issue Number>_<Description>
- Complete the development and test your changes.
- Add/update log messages for easier debugging.
- Add/update unit tests.
- Add/update documentation.
- Push local changes to GitHub.
- Submit a pull request to merge into develop.
Pull request:feature <Issue Number> <Description>
- Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues - Iterate until the reviewer(s) accept and merge your changes.
- Delete your fork or branch.
- Close this issue.