Update develop-ref after #3212 and #3143 - #3215
Merged
Merged
Conversation
* marine_and_cryosphere directories * fixing underlining * trying to fix warning msg. * trying to fix warning msg. again * Removing space before i.e. * updating medium_range and pbl directories * deleting empty file * removing lots of old text * precipitation files * removing junk file * removing extra # * add return * updating 3 dirs s2s, space_weather and unstructured_grids * updating short_range files * adding tc_and_extra_tc files * removing defunct -c option * added dropdown menus for marine_and_cryosphere * fixing problems and dropdowns for medium_range * fixing new problems * dropdowns for pbl and precip * adding double colons * Attempting to resolve errors * dropdowns for s2s and s2s_mjo * adding dropdowns for short_range files * final dropdown directories * Update docs/use_cases/model_applications/marine_and_cryosphere/GridStat_fcstRTOFS_obsGHRSST_climWOA_sst.py removing note Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * Apply suggestions from code review adding all of the suggestions Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * Update GridStat_fcstRTOFS_obsGHRSST_climWOA_sst.py ensuring literal include is in dropdown menu * Update GridStat_fcstRTOFS_obsOSTIA_iceCover.py ensuring literal include is in dropdown * ensuring literal include is in dropdown menu * marine_and_cryosphere MET Configuration dropdown fixes * remaining dropdown directories for MET Configuration * Julie changing dropdowns Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * HEAD needed to be removed * There are not MET tools used in this use case * There are no MET tools used in this use case * Apply suggestions from code review Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * Per #2741, few small changes * updates from pull request discussion * removing space Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * fixing dropdown Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * removing bash Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * update from Julie Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * Fixing error * lots of typos Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> * Adding period * Adding colon * Added period * Added period * Added period * typos and more Co-authored-by: Julie Prestopnik <jpresto@ucar.edu> --------- Co-authored-by: Julie Prestopnik <jpresto@ucar.edu>
* exclude metplus/scripts dir from PyCharm project to prevent incorrect duplicate code warnings with files that are sym linked * to expand on #2772, updating instructions to include note to set tmp directory for apptainer to prevent issues pulling large images that require a lot of temp space * per #511, add unit test for expected behavior to support comma-separated lists in a command line single config override that should fail until fix is made * add a test to ensure that the -c argument is properly ignored since it is can be used in old use cases that were created when the argument was required * per #511, add support for command line single config overrides to include values that are lists. Simplify logic to parse arguments to strip out -c/--config/-config arguments and skip check/error if argument is invalid because it is already handled in the metplus_config setup step that parses the arguments * added unit test to ensure that an invalid command line argument causes the appropriate failure from run_metplus.py
* adding In Memoriam section * Changing to "This Coordinated Release is dedicated to" removing version * adding links * fixing Randy's link * Per #2780, change case of words and add comma after year --------- Co-authored-by: Julie Prestopnik <jpresto@ucar.edu>
* per #2816, update title of workflow displayed in Actions tab to display the repo that triggered the workflow on workflow_dispatch events. Preserve the behavior of pull_request and push events by displaying the PR title or head commit message respectively * fix logic for run-name * Update event info to note that the workflow was triggered manually if the repo/title was not provided. Update description of workflow_dispatch argument to describe what it can be used for when triggering manually. Only run step to build URL to commit that triggered workflow if it was triggered from an external repo * add optional argument for workflow_dispatch to specify the title of the workflow run * revert change to repository argument description since title argument was added to handle custom workflow run titles
* per #2814, add documentation to describe how to set a single config variable on the command line * fixed typos and formatting issues * move new content to top of chapter and reword intro sentence * change label and add link to user env var section * per #2814, added info and examples for single config overrides with a list of values
… dev release notes. improve naming of drop downs for generating dev release notes to be consistent
…nder a category
…the RTD web interface
…TC web server to be done for the rc1 release instead of after the official release
* resolve some SonarQube complaints * per #2586, added function with tests to properly parse list of command line arguments that can now contain comma-separated lists that should not be split up into separate items * add support for {app}_{data_type}_FILE_WINDOW_BEGIN/END, e.g. GEN_VX_MASK_OBS_FILE_WINDOW_BEGIN. This just adds support for an additional variation of the config variable names * add support for an empty label for input templates * update wrapper to be consistent with other wrappers wrt finding input files, progress towards #2492. Allow file window range to be specified separately for mask and input files. Other cleanup to move towards consistent wrappers with fewer wrapper-specific overrides of functions like get_command * update unit tests to align with changes for #2492 * add documentation for config variables that are newly supported to allow file window range to be specified separately for mask and input files * renamed GEN_VX_MASK_OBS variables to be GEN_VX_MASK_INPUT as suggested by @JohnHalleyGotway in PR review * fix logic to properly read input files by handling inputs that support multiple inputs with labels (used by GridDiag and UserScript wrappers) and typical inputs (all other wrappers). Prior to this change only input templates that have the FCST or OBS identifier were read properly via get_input_templates
* Work in progress adding instructions dropdowns to the release guide * Per #2844, more dropdowns * Per #2844, add instruction dropdowns for all MET instructions * Per #2844, more dropdowns. * Per #2844, use instruction dropdowns throughout * Per #2844, remove METexpress AWS instruction since its no longer relevant. * Remove accidentally committed .DS_Store file.
…' based on 1/9/2025 decision at the METplus-Analysis Tools project meeting
…nd/or output formats.
…w reporting label and update scripts to run /bin/bash instead of /bin/sh, which now points to dash on seneca.
* Adding use case image * Adding MvMODE FireWx use case * Adding Fire Weather use cases * Updating use case image * Adding new use cases to the use case groups * Updating python embedding file * Updating documentation * Updating documentation and MTD use case * Adding HRRR use case * Adding documentation for Point-Stat use case * Updating use case groups, etc * Fixing some bad documentation links * Changing m/s to mph * Fixing weird link * Fixing file names * Adding some use case updates * Updating MvMODE case to use multiple thresholds * Updating images * Adding plotting yaml files * Fixing missing semicolon * Updating documentation * Fixing indent * Turning on one of the tests * MTD use case testing * Testing PointStat use case * Testing MvMODE use case * Removing CI override because it's behaving oddly * Updating MvMODE case to run all steps * Reducing automated testing for fire2 * Adding py_embed to use case group * Documentation corrections and updates * Fixing list error * Another documentation update * Documentation updates * Shortening testing for Point-Stat case * Changes * Updating contributors guide * Updating contributors guide * Updates * Updating a path to be more consistent * removing commented out line * Turning tests off for PR * SonarQube Sucks * Updating path * Fixed incorrect threshold * updated language, spelling --------- Co-authored-by: Christina Kalb <kalb@seneca.rap.ucar.edu> Co-authored-by: j-opatz <jopatz@ucar.edu>
…e or python embedding from check_for_python_embedding function because the function did multiple things and its purpose was confusing. The check_for_python_embedding function now only sets the file_type if the input is a python embedding script, the MET tool supports multiple python inputs, and the file_type was not explicitly specified by the user. The prevents an error because some tools require the file_type to be set for python embedding. Updated unit tests to properly test the two functions.
PR dtcenter/MET#3323 fixed ioda2nc to properly apply this threshold, which causes this setting to wipe out all values and ioda2nc fails. Removing this setting uses the new default value of NA, which should produce the same results as before this bugfix.
…_graphics that was deprecated and removed from MET via dtcenter/MET#3322.
…k and output_flag (#3201) * Per #3189, enhance PB2NC_QUALITY_MARK_THRESH to support thresholds instead of just integers. Added unit test to demonstrate newly supported format * Per #3189, add support for setting threshold values for IODA2NC_QUALITY_MARK_THRESH * Per #3194, update GridDiag wrapper's mask dictionary to support multiple masking regions * fix broken support for MET config dictionaries that don't have sub-directories * Per #3194, add output_flag to GridDiag * ignore all .idea files instead of certain files under the directory
…te MET configs with env vars substituted (#3210)
* Per #2570, add function handle_file_type to consistent handle setting file_type for fcst/obs/ens dictionaries. Added support for settings these for EnsembleStat and MTD wrappers. Added GRID_DIAG_FILE_TYPE variable as well * fix broken support for MET config dictionaries that don't have sub-directories * Per #2570, add docs and tests for new configs * use raw strings to prevent escape sequence warning on build doc script * Per PR #3203 review from @JohnHalleyGotway, update glossary entries to use recommended variable names and added missing item * remove duplicate entries
…e-Stat to 15 existing use cases to reproduce the same output that was previously being produced. (#3212)
* Per #3104, rename land_surface use case category to land * Per #3103, resolving RTD warning * To remove duplicate code, refactor 2 scripts into 1 with new command line argument to control which derivation to run * To remove duplicate code, refactor 2 obs scripts into 1 with new command line argument to control which derivation to run * update use case documentation to reflect changes to python embedding scripts --------- Co-authored-by: jprestop <jpresto@ucar.edu>
github-project-automation
Bot
moved this from 🩺 Needs Triage
to 🏁 Done
in METplus-13.0 Development
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
12 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#3212 added FULL output back and #3143 renamed land_surface use case category to land
Created by @georgemccabe