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.

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

Feature dtcenter/METplus-Internal#24 test failure #1895

Merged
merged 102 commits into from
Oct 26, 2022

Conversation

georgemccabe
Copy link
Collaborator

@georgemccabe georgemccabe commented Oct 25, 2022

GitHub issue is in internal repository: dtcenter/METplus-Internal#24

Changes

  • Per METplus-Internal 24, added unit tests to call the run_metplus.py script with different arguments and check that the return value to ensure that a failed use case returns a non-zero return code as expected.
  • Refactored the pytest logic to create a separate output directory for each test run and scrub that directory unless the test fails. Any pytest that uses the metplus_config pytest fixture to obtain a METplusConfig object with the minimum required settings will automatically remove the output directory created by that test unless one of the assert statements fail in the test. This makes it easier to review the logs for a failed test for debugging.

Pull Request Testing

  • Describe testing already performed for these changes:

Added the unit tests to satisfy the issue requirements

  • Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

Ensure that all of the automated tests pass. Optionally review the changes to the pytest fixtures in case the logic could be applied to other components.

  • Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes]

  • Do these changes include sufficient testing updates? [Yes]

  • Will this PR result in changes to the test suite? [No]

    New tests are added but they should all pass

  • Please complete this pull request review by 10/26/2022.

Pull Request Checklist

See the METplus Workflow for details.

  • Add any new Python packages to the METplus Components Python Requirements table.
  • Review the source issue metadata (required labels, projects, and milestone).
  • Complete the PR definition above.
  • Ensure the PR title matches the feature or bugfix branch name.
  • Define the PR metadata, as permissions allow.
    Select: Reviewer(s)
    Select: Organization level software support Project or Repository level development cycle Project
    Select: Milestone as the version that will include these changes
  • After submitting the PR, select Development with the original issue number. - listed above
  • After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
  • Close the linked issue and delete your feature or bugfix branch from GitHub.

…pper using STAT_ANALYSIS_JOB<n>, clean up indentation
…ead of using out-of-date file that should no longer be used
… to prevent tests from passing when they should not
…ots do not break before adding other logic to StatAnalysis wrapper
…r setting groups of list items that can be looped over, ci-run-diff
… from the FCST/OBS_VAR<n> variables and around level values read from FCST/OBS_LEVEL_LIST, ci-run-diff
…dds quotation marks to format the values instead of handling that later in the processing
…ort HHHMMSS (3 digits of hour). Updated tests to handle example from removed configuration file in internal/tests/pytests/plotting
…n each, add comments to describe the intended behavior
…lues into helper function to clean up logic and make it more readable, ci-run-diff
…ad offsets to be consistent with other wrappers, ci-run-diff
…ad offsets to be consistent with other wrappers, ci-run-diff
…ort very old wrapped MET config files for StatAnalysis
…hat was removing first letter of sentence for CUSTOM_LOOP_LIST
…put under METPLUS_TEST_OUTPUT_BASE and added logic to remove that directory before running tests. This prevents false success of tests that generate files and checks if they exist if they were generated from a previous successful run. Write to directory with hard-coded name to prevent accidentally scrubbing other files if the env var is not set properly
…lus.py script returns an expected failure return code
…fig file will automatically scrub the test output including logs and final conf only if the test succeeds. This makes it much easier to review test output to investigate issues. Tests that currently read in additional configuration files now call the metplus_config_files fixture to preserve the old behavior but those cases will not automatically clean up the output data. Eventually update those tests to set the config settings in the test instead of reading a separate file so that all tests that use a METplusConfig object will be scrubbed unless they fail
…d if the test passes, scrub files that must be generated using the old method, add documentation to pytest fixtures
@georgemccabe georgemccabe added this to the METplus-5.0.0 milestone Oct 25, 2022
@georgemccabe georgemccabe requested a review from bikegeek October 25, 2022 21:56
@bikegeek bikegeek marked this pull request as ready for review October 26, 2022 01:26
Copy link
Contributor

@bikegeek bikegeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All automated tests passed and all changes look good.

@georgemccabe georgemccabe merged commit 6fb3394 into develop Oct 26, 2022
@georgemccabe georgemccabe deleted the feature_24i_test_failure branch October 26, 2022 15:20
j-opatz added a commit that referenced this pull request Jun 7, 2023
* fixed TCPairs unit tests to handle default value for match_points

* bugfix #1691 remove whitespace from output file paths (#1721)

* added variables that are needed for use case and were accidentally removed

* log error and increment error count if required lat/lon bound variables are not set instad of calling sys.exit

* skip metplus_final.conf in check for new output to prevent false alarm diffs

* Update index.rst

replace METdatadb with METdataio

* Update metdatadb.rst

Replace metdatadb with metdataio

* renamed from metdatadb

* replacing all other instances of metdatdb/METdatadb with metdataio/METdataio

* after renaming directory from metdatadb to metdataio

* replace METdatadb with METdataio

* copied and replaced with metdataio filenames

* replaced with metdataio version

* Update the Release Guide while creating the MET-11.0.0-beta2 development release. Automatic Zenodo updates for each MET release have been DISABLED since they were being created for each development release. Revise the MET instructions to manually update the MET version on Zenodo only for official releases, not development or bugfix releases.

* Changed version number

* Added release notes

* Update the labels scripts by adding 1 new one and renaming metdatadb to metdataio.

* updated pandas version used for METdataio use cases to use 1.2.3 instead of 1.4.3

* update version to show development towards beta3 release

* Changed METdatadb references to METdataio

* Adding modulefiles for installation on various machines. (#1750)

* Feature 1746 recreate release (#1751)

* Adding instructions for recreating and existing release

* Reorder the step's based on suggestions from @georgemccabe

* Feature #1489 plot point obs wrapper (#1753)

* Fixing typo

* Feature #1657 clean up directory creation (#1756)

* Feature update labels (#1765)

* Modifications for METbaseimage

* Update README.md

Co-authored-by: johnhg <[email protected]>

* feature dtcenter/METplus-Internal#20  add user ID to log output (#1771)

* Per dtcenter/METplus-Internal#20, change log output to always include username and include user ID if it is available

* Per dtcenter/METplus-Internal#20, fixed bug introduced from commit 458714c to develop where error is thrown if username cannot be obtained, which occurs in environment running GHA tests

* Revert "Per dtcenter/METplus-Internal#20, fixed bug introduced from commit 458714c to develop where error is thrown if username cannot be obtained, which occurs in environment running GHA tests"

This reverts commit fb2d093.

* Revert "Per dtcenter/METplus-Internal#20, change log output to always include username and include user ID if it is available"

This reverts commit 458714c.

* per dtcenter/METplus-Internal#20, implemented working solution to user info logging

* change file path based on change to MODE output from dtcenter/MET#2251

* fix Contributor's Guide GitHub Workflow page (#1774)

* Feature dtcenter/METplus-Internal#32 add timestamp to final conf and file_lists (#1763)

* Feature dtcenter/METplus-Internal#15 checksum for release (#1769)

Co-authored-by: Julie Prestopnik <[email protected]>

* Feature dtcenter/METplus-Internal#32 change SCRUB_STAGING_DIR default and add documentation (#1784)

* fix to allow wildcard characters in UserScript input templates, ci-run-all-diff

* per #1787, allow NA instead of only integers for climo_mean/stdev hour_interval

* per #1787, added test for setting NA for climo_mean hour_interval

* Updating note format

* update Python version to 3.8.6

* Revert "update Python version to 3.8.6"

This reverts commit c1e6cbc.

* added return value checking to error and exit a non-zero value if docker build or docker push fail

* update instructions to find issues for a bugfix release (#1790)

* Feature dtcenter/METplus-Internal#34 set the log time to UTC/GMT (#1793)

* Feature #1330 MJO ENSO use case (#1794)

Co-authored-by: cristianastan2 <[email protected]>

* Feature #1816 ensemble vx removal from EnsembleStat (#1820)

Co-authored-by: John Halley Gotway <[email protected]>

* Feature #1816 rename cases and files (#1823)

Co-authored-by: John Halley Gotway <[email protected]>

* Move files under internal_tests into internal/tests (#1818)

* info for new use case

* renamed picture of unrelated use case to properly display

* Minor tweak to the release guide instructions for MET development releases.

* Feature #1817 5.0.0-beta3 release (#1825)

Co-authored-by: bikegeek <[email protected]>

* Update VERSION

* updated file name

* updated MET file locations

* explicitly set units for obs read via python embedding since PR dtcenter/MET#2288 no longer guesses units based on grib table units (which can be inaccurate of the actual data)

* turn off precip use case group

* Feature #1829 set unique ID in config (#1830)

* fixed typo in use case setting that causes differences in the test output

* Feature #1687 RuntimeFreq read only files needed for given run (#1841)

* Feature 1847 release (#1850)

* Per #1847, moved release notes to be its own chapter, reformatted component release notes, and added new section for upgrade instructions

* Per #1847, added release-notes internal tag

* Per #1847, fix error and reformat link

* Per #1847, modified release instructions to add a section about upgrade instructions

* Per #1847, modified release instructions to add a section about upgrade instructions

* Per #1847, resolve doc warnings

* Per #1847, updated instructions for clarity.

* Per #1847, modified text and link.

* Per #1847, modified wording and resolve error with note

* Feature #1863 fix GitHub Actions warnings (#1864)

* Feature #1874 prevent false diffs (#1875)

* Feature 1842 StatAnalysis - multiple issues (#1869)

* Bugfix #1884 develop PCPCombine {custom} in subtract method (#1887)

* Feature 1852 use case physics tendency (#1881)

* Issue #1852 initial work for the Planview plot

* Issue #1852 planview plot for physics tendency

* issue #1852 cleared away unused/unneccessary code

* Issue #1852 update script command

* Issue #1852 first draft, doesn't have correct content

* Issue #1852 updated name of output plot to include short_range in name

* Issue #1852 renamed to include fcstOnly label

* Issue #1852 initial commit

* Issue #1852

* Issue #1852

* Issue #1852 initial commit

* issue #1852 vertical cross section

* Issue 1852 clean up unused lines

* Issue #1852 initial commit

* Issue #1852

* Issue #1852 updates

* Issue #1852 fixed documentation warnings, but some problems remain

* fixing line length for chapter and section headers and one letter capitalization

* cleaning up line spacing and trying to fix a critical error.

* trying to fix critical error again

* Issue #1852 more description for data and dependency on METplotpy with additional instructions

* Issue #1852 update location of input data to single directory

* Removing duplicate files with a misformatted name that are not needed

* Per #1852, making changes to formatting to eliminate errors

* Issue #1852 remove redundant yaml config file

* Issue #1852 update PYTHONPATH information

* Issue #1852 fixed conflicts

* Issue #1852 update path to shapefile

* Issue #1852 fixed copy and paste error

* Issue #1852 add three use case entries: Planview, VerticalProfile, and VerticalCrossSection for the short range Physics tendency use case

* Issue #1852 add latest short range tests to top, move down list when everything passes

* Update use_case_groups.json

move short_range use cases #10-12 to below the short range test for #9

* Issue #1852 clean up conf files by providing variables for the input netCDF files and for the planview plot, allow user to indicate pressure levels

* Issue #1852 Indicate external Python packages

* Issue #1852 added external dependency section

* removed env vars that don't need to be set -- METPLOTPY_BASE is set by the automated test conf (.github/parm/test_settings.conf) and PYTHONPATH doesn't need to be set because METplotpy is installed in Python environment for use case tests

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_Planview.py

Fixed one typo

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalCrossSection.py

Fixed a typo

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalProfile.py

Same typo, same fix

* Issue #1852 remove extraneous entry for USER_SCRIPT_COMMAND, a hold-over from the original implementation which doesn't allow the user to easily specify the FV3 history file and grid spec netCDF file

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalProfile.conf

Removed the incorrect USER_SCRIPT_COMMAND in previous commit.  Reinstated the correct version and removed the incorrect command.

* update html file from Planview to VerticalProfile

Co-authored-by: Lisa Goodrich <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
Co-authored-by: George McCabe <[email protected]>
Co-authored-by: Hank Fisher <[email protected]>

* Feature #1893 StatAnalysis custom looping (#1894)

* Feature dtcenter/METplus-Internal#24 test failure (#1895)

* updated version for creating release (#1897)

* update release number for development towards beta5

* Adding one sentence to about daily follow up on discussions.

* initial changes for new use case

* updated paths, documentation

* updated automated testing files

* Feature #344 rearrange utils (#1900)

* first additions for new use case, corrected documentation for old use case

* Move Docker scripts to be consistent with other METplus components (#1907)

* added picture

* Correct spelling mistake

* handle if input to format_thresh is already a list

* Revert "handle if input to format_thresh is already a list"

This reverts commit d329275.

* handle if input to format_thresh is already a list

* per #1910, added time util function to add field info to time info dictionary and updated wrapper logic to use this function to add this info instead of passing field info (var_info) to CommandBuilder.find_data to get level info, ci-run-all-diff

* change logic to preserve full level string for [fcst/obs]_level so SeriesAnalysis cases build the same file names, but parse out time info for setting generic level for string sub to preserve previous behavior

* try setting all input values in output dictionary, ci-run-all-diff

* removed commented code

* add quotes around input files to plot_point_obs command to allow support for Python Embedding as input

* put all arguments on single line

* remove commented code

* per #1910, added unit tests to ensure that level is set properly in the forecast input template

* added script to create geovista conda environment via Docker to use in the automated use case tests

* Feature 1561 use case ugrid python embedding (#1927)

Co-authored-by: George McCabe <[email protected]>

* Feature 1453 use case iodav2 python embedding (#1930)

Co-authored-by: George McCabe <[email protected]>

* fix permissions for script that were somehow changed to be not executable

* Bugfix #1939 develop - failure reading obs when zipped file also exists (#1941)

* Feature #1819 alert users if updates to EnsembleStat use cases are needed (#1945)

Co-authored-by: Dan Adriaansen <[email protected]>

* fixed underline that did not match title

* Feature #1898 tc new options (#1932)

close undefined

* fixed broken doc util logic from bad PR merge

* Feature #1569 EnsembleStat -ens_mean argument (#1952)

* Feature #1953 seeps config in GridStat and PointStat (#1954)

Co-authored-by: johnhg <[email protected]>

* Feature #1566 update Python version to 3.8.6 (#1958)

* Feature #1956 5.0.0-beta4 release (#1963)

* Update VERSION

* added warning in release notes that location of METplus Dockerfile has changed

* combined release notes for all betas for the v5.0.0 release

* updated tags for each METplus component to match the upcoming release

* Feature #1885 gh_projects (#1973)

Co-authored-by: George McCabe <[email protected]>

* Switch from double to single quotes.

* Update formatting of the GitHub project filters so that they can be copy/pasted directly from RTD into the GitHub project filter.

* Feature #1953 PointStat output_flag.seeps and output_flag.seeps_mpr (#1976)

* Feature #1711, #1783, #1814, and #1835 Documentation - Multiple files, PCPCombine constant init, Docker instructions, and MET links (#1967)

Co-authored-by: Dan Adriaansen <[email protected]>

* run diff logic on push for develop and main_v* branches, fix logic to properly run all use cases

* fixed name of external

* added Bugfix: to default title of new bugfix issue

* added space after New Use Case:

* changed references to METdatadb to METdataio that were missed

* Update overview.rst

Removed statsmodels from the table, this package is no longer a requirement.  The necessary functionality was pulled out of the statsmodels repository and incorporated into the METcalcpy repository

* Update overview.rst

METdataio *must* only use pandas 1.2.3.  METdataio also requires pyyaml.

* Update overview.rst

METdatio uses pytest to test aspects of the METreformat module

* Update rotate_authorship.rst

Explicitly add the step to commit and push changes to the file containing the author list.

* Update overview.rst

Added METdataio for lxml, numpy, pandas, and pymysql

* Update finalize_release_on_github_official.rst

added branch protection rules step and the interface to Github was modified, you need to click on the stacked '->' icon to switch default branches

* CI: replace / with _ to prevent errors creating docker tags for tests

* Update overview.rst

included METdataio for python 3.8.6+ requirement

* Update overview.rst

psutil no longer needed in METplotpy and METcalcpy

* Update overview.rst

METcalcpy no longer requires imutils and allow versions 0.5.3 and above

* Update overview.rst

cleaned up formatting for imutils

* change asterisk to double asterisk to match branches that include forward slash /

* update version of certifi based on dependabot auto-generated PR #1982

* Feature 1819 format doc (#1981)

Co-authored-by: George McCabe <[email protected]>

* Update use case keywords (#1984)

* Feature #1980 Use the main_vX.Y of METplus components for main METplus tests (#1985)

* Feature #1986 v5.0.0 release (#1987)

Closes #1986

* updated version for development towards beta1 of 5.1.0

* allow rest of workflow to run even if job to update data volumes fails in case server is down

* updated version for action to update input data volumes and added max number of DockerHub tag pages to search to 15

* updated release notes for official METplus release to untar the sample data for next version directory

* Update release guide to create the input test data directory for official MET releases.

* Follow the METplus naming conventions for updating the the DTC web server data.

* install library needed for geovista package, remove editable flag so geovista code will be added directly to python instead of reading it from the source code location, ci-skip-all

* Feature 685 log updates (#1992)

* add script to create conda environment with MetPy 1.4 for SWPC use case

* Bugfix develop Fix broken documentation links (#2004)

* fixed bug to set environment variables in the RegridDataPlane environment instead of the PyEmbedIngest environment

* Feature #2015 modulefiles (#2024)

* Adding modulefile for WCOSS2

* Updated references of 4.1.1 to 5.0.0

* Feature #2005 updates to create v5.1 conda environments that use debian10 and Python 3.10.4 (#2043)

* update versions to fix bugs for METplotpy/calcpy use cases

* Bugfix #2026 develop StatAnalysis looping (#2028)

* Feature #2022 update python version to 3.10.4 for repo files (#2047)

* Feature #2022 Use Debian 10 / Python 3.10.4 in automated tests (#2050)

* add use case group name to output path so that output and error logs will contain use case category to more easily see which case was run

* remove -group0 from group names for output path

* add manual workflow to update truth data

* ci: set gh token on checkout to enable push, ci-skip-all

* Set user name/email via git config to allow push

* add token to checkout

* Feature #2049 Multiple interp.type.method/width (#2051)

* added missing argument to script if building envs locally

* get main_vX.Y truth data if running on main_vX.Y branch

* Updating README.md for label creation and adding bold to common_labels.txt

* Updated color for bold label

* Feature #2054 consensus.write_members in TCPairs (#2057)

* Create LICENSE.md (#2061)

* per #2006, fix priority of obs_window config variables so that wrapper-specific version is preferred over generic OBS_WINDOW_BEGIN/END (#2062)

* Feature 2065 v5.1.0 beta1 (#2066)

* updated version number

* added release notes

* Update docs/Users_Guide/release-notes.rst

Co-authored-by: John Halley Gotway <[email protected]>

---------

Co-authored-by: John Halley Gotway <[email protected]>

* updated version with -dev

* adding sphinx design to 3 files so dropdown menus will work in the release notes.

* fix version numbers for python requirements for building documentation

* fixed tab vs spaces

* Bugfix #2070 var list numeric order (#2072)

* Adding files for jet and hera

* added commented process list that was accidentally removed -- this is used if the pre-processing steps are needed

* Bugfix #2087 develop docs_pdf (#2091)

* Feature 2076 release dropdown menu (#2083)

* test dropdowns

* Update release-notes.rst

Attempting to resolve WARNING

* testing dropdowns

* adding dropdown menus

* testing dropdown

remove from warning section

* dropdown testing

* dropdown menu tests

* dropdown menus

after the warnings section, the dropdown menu only has one space instead of the normal 2 spaces.  If it has 2 spaces, it gets sucked into the warning section.

* spaces with warning

* no spaces for warning

* Update release-notes.rst

warning note has to be indented 1 space.  Originally it was no spaces but then all of the dropdown menus end up in the warning message.  If it's 2 spaces, as is normal, then the warning is indented, which is unnecessary.  1 space is slightly indented and keeps the dropdown menus separate.

---------

Co-authored-by: jprestop <[email protected]>

* Bugfix #2096/#2098 develop - fix skip if output exists and do not error if no commands were run (#2099)

* Feature 1516 use case mvmode (#2094)

Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: George McCabe <[email protected]>

* Fix for Dockerfile smell DL4000 (#2112)

Co-authored-by: Giovanni Rosa <[email protected]>

* Changed "PROJECT" to "CYCLE" ASSIGNMENT

* Feature 2115 update use cases (#2133)

* Bugfix #2082 develop regrid.convert/censor_thresh/censor_val (#2140)

fix #2082 main_v5.0 regrid.convert/censor_thresh/censor_val (#2101)

* Bugfix #2137 develop PointStat -obs_valid_beg/end (#2141)

* Feature #2143 5.1.0-beta2 release (#2144)

* update release guide steps with new info based on changes to GitHub web interface

* update version for development towards rc1 release

* Feature #1977 ARGO use case (#2148)

Co-authored-by: j-opatz <[email protected]>

* Feature develop cycle change (#2150)

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* Added paths to ignore

---------

Co-authored-by: jprestop <[email protected]>

* Documentation: Update Truth Data instructions (#2154)

* upgrade python to 3.10 for documentation builds to fix failured introduced by urllib3 (see urllib3/urllib3#2168)

* update workflow to update input test data to use branch from pull down or enter branch by hand if branch does not exist in dtcenter/METplus repo, e.g. if updating data for a PR coming from a fork

* Bugfix #2161 develop PCPCombine additional field arguments in -subtract mode (#2162)

* changed order of workflow event triggering info to more easily see the repository and commit sha that triggered the workflow

* fix docker compose install and call -- needed due to new Alpine Linux release 3.18.0 that does not include docker-compose in the apk repository

* Bugfix #2168 develop - StatAnalysis time shift (#2169)

* Feature 1978 update pyreq doc (#2177)

* Updating for 5.1.0

* Adding greater than or equal to signs

* Feature 2106 release support (#2182)

* Added a paragraph for Release Support Policy

* Added a sentence about contacting the team through the METplus GitHub Discussions Forum.

* Added Release Support Policy info

* Per #2159, modify instructions to reflect that we only want a coordinated release for the official releases and updates to the coordinated release section for the bugfix releases. (#2183)

* Feature 2147 workflow doc (#2185)

* Add new images for github workflow

* Added PR section

* Modified the Find the GitHub Issue to be Find or Create a GitHub issue

* Trying out image keyword compared with figure keyword

* Trying out various widths

* Added new figures

* Reverting to figures from images after modifying figures

* Adding the old images which are better quality

* Trying out new indention

* Working on formatting again

* Reducing image widths

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update github_workflow.rst

---------

Co-authored-by: George McCabe <[email protected]>

* added scripts to create new conda environment for PANDA-C use cases

* Bugfix #2189 develop - spaces in complex thresholds (#2191)

* added release notes for completed issues for rc1

* Feature 2011 tci from cesm fluxnet2015 netcdf (#2192)

Co-authored-by: Mrinal Biswas <[email protected]>
Co-authored-by: George McCabe <[email protected]>

* Bugfix #2179 develop TCPairs fix -diag argument (#2187)

* sort list of files to make command more readable and make logic more easily testable

* fix unit test to expect alphabetical order of files

* Feature #2197 MODE new multi-variate options (#2203)

* added to release notes

* added release notes for other items completed for rc1 release

* Feature #1626 TCDiag Wrapper (#2201)

* Feature #1610 first round of SonarQube fixes (#2200)

* Feature #2195 create v5.1.0-rc1 release (#2204)

* Update version for development towards official release

* updated script to use metplotpy as base and install pygrib because conda was failing trying to install matplotlib in Docker for some strange reason

* Feature 2188 use case panda c (#2202)

* added first use case, need to finish docs and add to list

* added documentation files, 3 new cases

* added 5th use case, docs

* final use case, corrected paths for output

* Update GridStat_fcstGFS_obsERA5_lowAndTotalCloudFrac.py

* removed chars, added space to last section

---------

Co-authored-by: George McCabe <[email protected]>
Co-authored-by: bikegeek <[email protected]>
Co-authored-by: bikegeek <[email protected]>
Co-authored-by: John Halley Gotway <[email protected]>
Co-authored-by: Hank Fisher <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
Co-authored-by: cristianastan2 <[email protected]>
Co-authored-by: j-opatz <[email protected]>
Co-authored-by: Lisa Goodrich <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
Co-authored-by: j-opatz <[email protected]>
Co-authored-by: Will Mayfield <[email protected]>
Co-authored-by: Dan Adriaansen <[email protected]>
Co-authored-by: lisagoodrich <[email protected]>
Co-authored-by: metplus-bot <[email protected]>
Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: Giovanni Rosa <[email protected]>
Co-authored-by: Giovanni Rosa <[email protected]>
Co-authored-by: mrinalbiswas <[email protected]>
Co-authored-by: Mrinal Biswas <[email protected]>
georgemccabe added a commit that referenced this pull request Sep 19, 2023
* Feature dtcenter/METplus-Internal#24 test failure (#1895)

* updated version for creating release (#1897)

* update release number for development towards beta5

* Adding one sentence to about daily follow up on discussions.

* initial changes for new use case

* updated paths, documentation

* updated automated testing files

* Feature #344 rearrange utils (#1900)

* first additions for new use case, corrected documentation for old use case

* Move Docker scripts to be consistent with other METplus components (#1907)

* added picture

* Correct spelling mistake

* handle if input to format_thresh is already a list

* Revert "handle if input to format_thresh is already a list"

This reverts commit d329275.

* handle if input to format_thresh is already a list

* per #1910, added time util function to add field info to time info dictionary and updated wrapper logic to use this function to add this info instead of passing field info (var_info) to CommandBuilder.find_data to get level info, ci-run-all-diff

* change logic to preserve full level string for [fcst/obs]_level so SeriesAnalysis cases build the same file names, but parse out time info for setting generic level for string sub to preserve previous behavior

* try setting all input values in output dictionary, ci-run-all-diff

* removed commented code

* add quotes around input files to plot_point_obs command to allow support for Python Embedding as input

* put all arguments on single line

* remove commented code

* per #1910, added unit tests to ensure that level is set properly in the forecast input template

* added script to create geovista conda environment via Docker to use in the automated use case tests

* Feature 1561 use case ugrid python embedding (#1927)

Co-authored-by: George McCabe <[email protected]>

* Feature 1453 use case iodav2 python embedding (#1930)

Co-authored-by: George McCabe <[email protected]>

* fix permissions for script that were somehow changed to be not executable

* Bugfix #1939 develop - failure reading obs when zipped file also exists (#1941)

* Feature #1819 alert users if updates to EnsembleStat use cases are needed (#1945)

Co-authored-by: Dan Adriaansen <[email protected]>

* fixed underline that did not match title

* Feature #1898 tc new options (#1932)

close undefined

* fixed broken doc util logic from bad PR merge

* Feature #1569 EnsembleStat -ens_mean argument (#1952)

* Feature #1953 seeps config in GridStat and PointStat (#1954)

Co-authored-by: johnhg <[email protected]>

* Feature #1566 update Python version to 3.8.6 (#1958)

* Feature #1956 5.0.0-beta4 release (#1963)

* Update VERSION

* added warning in release notes that location of METplus Dockerfile has changed

* combined release notes for all betas for the v5.0.0 release

* updated tags for each METplus component to match the upcoming release

* Feature #1885 gh_projects (#1973)

Co-authored-by: George McCabe <[email protected]>

* Switch from double to single quotes.

* Update formatting of the GitHub project filters so that they can be copy/pasted directly from RTD into the GitHub project filter.

* Feature #1953 PointStat output_flag.seeps and output_flag.seeps_mpr (#1976)

* Feature #1711, #1783, #1814, and #1835 Documentation - Multiple files, PCPCombine constant init, Docker instructions, and MET links (#1967)

Co-authored-by: Dan Adriaansen <[email protected]>

* run diff logic on push for develop and main_v* branches, fix logic to properly run all use cases

* fixed name of external

* added Bugfix: to default title of new bugfix issue

* added space after New Use Case:

* changed references to METdatadb to METdataio that were missed

* Update overview.rst

Removed statsmodels from the table, this package is no longer a requirement.  The necessary functionality was pulled out of the statsmodels repository and incorporated into the METcalcpy repository

* Update overview.rst

METdataio *must* only use pandas 1.2.3.  METdataio also requires pyyaml.

* Update overview.rst

METdatio uses pytest to test aspects of the METreformat module

* Update rotate_authorship.rst

Explicitly add the step to commit and push changes to the file containing the author list.

* Update overview.rst

Added METdataio for lxml, numpy, pandas, and pymysql

* Update finalize_release_on_github_official.rst

added branch protection rules step and the interface to Github was modified, you need to click on the stacked '->' icon to switch default branches

* CI: replace / with _ to prevent errors creating docker tags for tests

* Update overview.rst

included METdataio for python 3.8.6+ requirement

* Update overview.rst

psutil no longer needed in METplotpy and METcalcpy

* Update overview.rst

METcalcpy no longer requires imutils and allow versions 0.5.3 and above

* Update overview.rst

cleaned up formatting for imutils

* change asterisk to double asterisk to match branches that include forward slash /

* update version of certifi based on dependabot auto-generated PR #1982

* Feature 1819 format doc (#1981)

Co-authored-by: George McCabe <[email protected]>

* Update use case keywords (#1984)

* Feature #1980 Use the main_vX.Y of METplus components for main METplus tests (#1985)

* Feature #1986 v5.0.0 release (#1987)

Closes #1986

* updated version for development towards beta1 of 5.1.0

* allow rest of workflow to run even if job to update data volumes fails in case server is down

* updated version for action to update input data volumes and added max number of DockerHub tag pages to search to 15

* updated release notes for official METplus release to untar the sample data for next version directory

* Update release guide to create the input test data directory for official MET releases.

* Follow the METplus naming conventions for updating the the DTC web server data.

* install library needed for geovista package, remove editable flag so geovista code will be added directly to python instead of reading it from the source code location, ci-skip-all

* Feature 685 log updates (#1992)

* add script to create conda environment with MetPy 1.4 for SWPC use case

* Bugfix develop Fix broken documentation links (#2004)

* fixed bug to set environment variables in the RegridDataPlane environment instead of the PyEmbedIngest environment

* Feature #2015 modulefiles (#2024)

* Adding modulefile for WCOSS2

* Updated references of 4.1.1 to 5.0.0

* Feature #2005 updates to create v5.1 conda environments that use debian10 and Python 3.10.4 (#2043)

* update versions to fix bugs for METplotpy/calcpy use cases

* Bugfix #2026 develop StatAnalysis looping (#2028)

* Feature #2022 update python version to 3.10.4 for repo files (#2047)

* Feature #2022 Use Debian 10 / Python 3.10.4 in automated tests (#2050)

* add use case group name to output path so that output and error logs will contain use case category to more easily see which case was run

* remove -group0 from group names for output path

* add manual workflow to update truth data

* ci: set gh token on checkout to enable push, ci-skip-all

* Set user name/email via git config to allow push

* add token to checkout

* Feature #2049 Multiple interp.type.method/width (#2051)

* added missing argument to script if building envs locally

* get main_vX.Y truth data if running on main_vX.Y branch

* Updating README.md for label creation and adding bold to common_labels.txt

* Updated color for bold label

* Feature #2054 consensus.write_members in TCPairs (#2057)

* Create LICENSE.md (#2061)

* per #2006, fix priority of obs_window config variables so that wrapper-specific version is preferred over generic OBS_WINDOW_BEGIN/END (#2062)

* Feature 2065 v5.1.0 beta1 (#2066)

* updated version number

* added release notes

* Update docs/Users_Guide/release-notes.rst

Co-authored-by: John Halley Gotway <[email protected]>

---------

Co-authored-by: John Halley Gotway <[email protected]>

* updated version with -dev

* adding sphinx design to 3 files so dropdown menus will work in the release notes.

* fix version numbers for python requirements for building documentation

* fixed tab vs spaces

* Bugfix #2070 var list numeric order (#2072)

* Adding files for jet and hera

* added commented process list that was accidentally removed -- this is used if the pre-processing steps are needed

* Bugfix #2087 develop docs_pdf (#2091)

* Feature 2076 release dropdown menu (#2083)

* test dropdowns

* Update release-notes.rst

Attempting to resolve WARNING

* testing dropdowns

* adding dropdown menus

* testing dropdown

remove from warning section

* dropdown testing

* dropdown menu tests

* dropdown menus

after the warnings section, the dropdown menu only has one space instead of the normal 2 spaces.  If it has 2 spaces, it gets sucked into the warning section.

* spaces with warning

* no spaces for warning

* Update release-notes.rst

warning note has to be indented 1 space.  Originally it was no spaces but then all of the dropdown menus end up in the warning message.  If it's 2 spaces, as is normal, then the warning is indented, which is unnecessary.  1 space is slightly indented and keeps the dropdown menus separate.

---------

Co-authored-by: jprestop <[email protected]>

* Bugfix #2096/#2098 develop - fix skip if output exists and do not error if no commands were run (#2099)

* Feature 1516 use case mvmode (#2094)

Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: George McCabe <[email protected]>

* Fix for Dockerfile smell DL4000 (#2112)

Co-authored-by: Giovanni Rosa <[email protected]>

* Changed "PROJECT" to "CYCLE" ASSIGNMENT

* Feature 2115 update use cases (#2133)

* Bugfix #2082 develop regrid.convert/censor_thresh/censor_val (#2140)

fix #2082 main_v5.0 regrid.convert/censor_thresh/censor_val (#2101)

* Bugfix #2137 develop PointStat -obs_valid_beg/end (#2141)

* Feature #2143 5.1.0-beta2 release (#2144)

* update release guide steps with new info based on changes to GitHub web interface

* update version for development towards rc1 release

* Feature #1977 ARGO use case (#2148)

Co-authored-by: j-opatz <[email protected]>

* Feature develop cycle change (#2150)

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* change PROJECT ASSIGNMENT to CYCLE ASSIGNMENT

* Added paths to ignore

---------

Co-authored-by: jprestop <[email protected]>

* Documentation: Update Truth Data instructions (#2154)

* upgrade python to 3.10 for documentation builds to fix failured introduced by urllib3 (see urllib3/urllib3#2168)

* update workflow to update input test data to use branch from pull down or enter branch by hand if branch does not exist in dtcenter/METplus repo, e.g. if updating data for a PR coming from a fork

* Bugfix #2161 develop PCPCombine additional field arguments in -subtract mode (#2162)

* changed order of workflow event triggering info to more easily see the repository and commit sha that triggered the workflow

* fix docker compose install and call -- needed due to new Alpine Linux release 3.18.0 that does not include docker-compose in the apk repository

* Bugfix #2168 develop - StatAnalysis time shift (#2169)

* Feature 1978 update pyreq doc (#2177)

* Updating for 5.1.0

* Adding greater than or equal to signs

* Feature 2106 release support (#2182)

* Added a paragraph for Release Support Policy

* Added a sentence about contacting the team through the METplus GitHub Discussions Forum.

* Added Release Support Policy info

* Per #2159, modify instructions to reflect that we only want a coordinated release for the official releases and updates to the coordinated release section for the bugfix releases. (#2183)

* Feature 2147 workflow doc (#2185)

* Add new images for github workflow

* Added PR section

* Modified the Find the GitHub Issue to be Find or Create a GitHub issue

* Trying out image keyword compared with figure keyword

* Trying out various widths

* Added new figures

* Reverting to figures from images after modifying figures

* Adding the old images which are better quality

* Trying out new indention

* Working on formatting again

* Reducing image widths

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update docs/Contributors_Guide/github_workflow.rst

Co-authored-by: George McCabe <[email protected]>

* Update github_workflow.rst

---------

Co-authored-by: George McCabe <[email protected]>

* added scripts to create new conda environment for PANDA-C use cases

* Bugfix #2189 develop - spaces in complex thresholds (#2191)

* added release notes for completed issues for rc1

* Feature 2011 tci from cesm fluxnet2015 netcdf (#2192)

Co-authored-by: Mrinal Biswas <[email protected]>
Co-authored-by: George McCabe <[email protected]>

* Bugfix #2179 develop TCPairs fix -diag argument (#2187)

* sort list of files to make command more readable and make logic more easily testable

* fix unit test to expect alphabetical order of files

* Feature #2197 MODE new multi-variate options (#2203)

* added to release notes

* added release notes for other items completed for rc1 release

* Feature #1626 TCDiag Wrapper (#2201)

* Feature #1610 first round of SonarQube fixes (#2200)

* Feature #2195 create v5.1.0-rc1 release (#2204)

* Update version for development towards official release

* updated script to use metplotpy as base and install pygrib because conda was failing trying to install matplotlib in Docker for some strange reason

* Feature 2188 use case panda c (#2202)

* added first use case, need to finish docs and add to list

* added documentation files, 3 new cases

* added 5th use case, docs

* final use case, corrected paths for output

* Update GridStat_fcstGFS_obsERA5_lowAndTotalCloudFrac.py

* removed chars, added space to last section

* turn off 5/6 new air quality and comp use cases because they are causing the disk to fill up in the automated tests and only 1 is needed to test the new behavior. Reordered use case groups to be alphabetical

* disabled other new air quality use case because the disk is still filling up from this case

* Feature 2136 cross spectra (#2208)

* per #2212, continue instead of return so other forecast leads will not be skipped

* Feature #2210 MODE multi-variate variable rename (#2211)

* Bugfix diff tests (#2217)

* added change from NCEP produtil that was somehow not included in the changes pulled from their repo before starting SonarQube work

* Feature #2228 v5.1.0-rc2 release (#2229)

* update version after rc2 release to note development towards official 5.1.0 release

* Feature #2188 move PANDA-C cases to clouds category and fix automated tests (#2237)

Co-authored-by: j-opatz <[email protected]>

* Feature 2198 use case amdar pbl (#2206)

Co-authored-by: Daniel Adriaansen <[email protected]>
Co-authored-by: George McCabe <[email protected]>

* Bugfix #2235 rename multivar_itensity to multivar_intensity_flag (#2236)

* Feature #2230 MvMODE usecase doc updates (#2239)

This updates the use-case documentation to go along with the multivariate MODE enhancements.

* combine release notes from development releases for 5.1.0 release

* fix indentation to prevent incorrect nesting of drop down menus

* Bugfix #2241 Create directory containing -out_stat file (#2242)

* Enhance diff logic to control decimal precision to prevent failures (#2243)

* Bugfix #2245 use unique run ID to name logger instance (#2247)

* per #2245, add METplusConfig class function that is called when object is deleted to close log handlers. This prevents OSError: [Errno 24] Too many open files from running all pytests

* Feature #1626 TCDiag updates to wrapper and basic use case (#2248)

Co-authored-by: Jonathan Vigh <[email protected]>

* move new use case into another group

* Bugfix #2244 develop fix diff tests (#2254)

* skip diff for specific png images that produce diffs occasionally but look the same visually

* add 2 more keywords for use cases that occasionally flag small, unnoticeable diffs in png images

* Feature #2253 update tests for diff_util (#2256)

Co-authored-by: root <root@localhost>

* Feature #1974 Documentation: update coding standards section (#2260)

* Feature #2253 conftest fixture to set pytest tmpdir (#2261)

* Feature #2151 Documentation: Add quick search keywords METplotpy/calcpy use cases (#2259)

* Feature #2138 CI enhance MET Docker tag override (#2258)

* Updating development instructions.

* fixed typo

* fixed another typo

* Feature #2207 Documentation - Updates to Contributor's Guide (#2263)

Co-authored-by: Dan Adriaansen <[email protected]>

* Feature develop update install files (#2266)

* Updating installation files

* Removing run_commands.sh

* Updated the file to add a MINICONDA_PATH

* Adding files for jet

* Adding orion file

* Removing beta references

* Corrected met version

* Adding files for WCOSS2 machines

* Removing rc1 from acorn file

* Removing beta1

* Bugfix #1853 develop - PointStat don't require mask variables to be set (#2262)

* update version of certifi to match auto-generated dependabot PR #2267

* feature #2253 more tests for diff_util.py (#2264)

* update versions of METplus components for the next official release

* updated release notes for 5.1.0 release

* removed duplicate entries in release notes

* Added EC2 instance recipee with S3 mounting ability (#2269)

* Updating the Subsetting Tests by Marker section (#2272)

* update release date for 5.1.0 release (#2276)

* update version for development towards 6.0.0-beta1 release

* added new use case that was missing

* Bugfix #2279 develop - buoy station file from 2022 (#2280)

* Feature 2253 series analysis tests (#2277)

* Fix typo in pull_request_template.md

* Make code coverage report available (#2287)

* Use updated dtcenter/metplus-action-data-update tag that fixes bug that causes GHA disk to exceed capacity when too many data volumes are created. Use specific commit of coveralls python action to ensure it will not change

* removed climatology keyword from use cases that aren't relevant that were accidentally added with PR #1984

* update readme with current information

* Feature #2282 coord_release_docs (#2288)

Co-authored-by: George McCabe <[email protected]>

* continue workflow if coveralls step fails -- this typically happens on a pull request from a fork because the authentication fails

* Feature 2253 system util tests (#2297)

* Feature #2294 LSR use case (#2295)

* Feature 2253 run util tests (#2307)

* Release Guide - remove beta/rc tags/releases (#2309)

* Add 'component: repository maintenance' label. Already ran the script to push this change and the NOAA OAR reporting label to all the METplus repos.

* Hotfix for labels, adding a new one for 'component: input data' and also fixing the get_lablels.sh to search for up to 200 existing labels. Also work on the log messages.

* Fix typo in comment

* Feature #2283 time looping consolidation (#2311)

Co-authored-by: John Halley Gotway <[email protected]>

* New issue template: Update Truth (#2332)

Co-authored-by: John Halley Gotway <[email protected]>

* feature #2253 tests for run_util, 'Usage' bugfix (#2313)

* Feature #2338 Debian 12 Conda environments (#2341)

Co-authored-by: John Halley Gotway <[email protected]>

* Feature 1488 usecase satwinds (#2324)

Co-authored-by: George McCabe <[email protected]>

* Feature #2283 include times (#2345)

Co-authored-by: John Halley Gotway <[email protected]>

* feature #2253 tests for config_validate, mock logger in conftest (#2320)

* Feature #2299 / #2310 Remove deprecated MET config env vars and TCMPRPlotter (#2347)

Co-authored-by: John Halley Gotway <[email protected]>

* remove MODEL and OBTYPE to use defaults from MET config

* Feature #2348 v6.0.0-beta1 release (#2351)

* update version for development towards 6.0.0-beta2 release

* copy libGL and libEGL files into docker image to prevent errors with geovista python package

* run use that uses geovista to test

---------

Co-authored-by: George McCabe <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
Co-authored-by: j-opatz <[email protected]>
Co-authored-by: j-opatz <[email protected]>
Co-authored-by: Hank Fisher <[email protected]>
Co-authored-by: Will Mayfield <[email protected]>
Co-authored-by: Dan Adriaansen <[email protected]>
Co-authored-by: johnhg <[email protected]>
Co-authored-by: bikegeek <[email protected]>
Co-authored-by: lisagoodrich <[email protected]>
Co-authored-by: metplus-bot <[email protected]>
Co-authored-by: Lisa Goodrich <[email protected]>
Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: Tracy Hertneky <[email protected]>
Co-authored-by: Giovanni Rosa <[email protected]>
Co-authored-by: Giovanni Rosa <[email protected]>
Co-authored-by: mrinalbiswas <[email protected]>
Co-authored-by: Mrinal Biswas <[email protected]>
Co-authored-by: Christina Kalb <[email protected]>
Co-authored-by: jason-english <[email protected]>
Co-authored-by: Jonathan Vigh <[email protected]>
Co-authored-by: John Sharples <[email protected]>
Co-authored-by: root <root@localhost>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants