-
Notifications
You must be signed in to change notification settings - Fork 38
Update develop-ref after #2989 #3000
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
Merged
Merged
Conversation
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
…g of params, and also switches the fluxnet metadata file to a command line argument instead of an environment variable.
…d filename patterns, DEBUG mode added and set to False by default, adjustment of print statements for logging, and refactoring filtering of stations to ensure we don't process a file that we shouldn't by better coupling of filenames and stations.
…s comments for use case changes.
…e group from running to easily disable it temporarily. Disable short_range:14 use case until it can be fixed with #2551
…ESM_obsFLUXNET2015_TCI.py Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com>
Feature 2388 update tci
… numpy masked values (MaskedConstant)
* first pass over doc, basic files. Needs config update and finalize doc * add rioxarray conda environment * update out-of-date info * added image, updated docs, attempt to resolve compile issue * removing space for error * updated remaining files, adjusted docs * fix incorrect naming for another use case that had errors in use case doc rendering * fix typos and formatting * fix path to thumbnail image * add py_embed keyword to tell automated test suite to set MET_PYTHON_EXE to the version of python in the rioxarray conda env * fix SonarQube complaint "Fix this attribute access on a value that can be 'None'." despite the function cannot return None because it will exit the script if an invalid input is provided --------- Co-authored-by: George McCabe <23407799+georgemccabe@users.noreply.github.com>
* per #2962, begin implementation of RMWAnalysis wrapper * start basic use case conf * read BOTH_VAR<n> variables into data field * change pycharm python interpreter * ignore file generated by building docs locally * update use case to read input data * Per #2962, add documentation, unit tests, and basic use case for RMWAnalysis wrapper * per #2962, added thumbnail for use case documentation * fix incorrect formatting * per feedback from @JohnHalleyGotway, change 5 config variables to support lists of values instead of a single string
* per #2890, refactor Example wrapper to support all EXAMPLE_RUNTIME_FREQ settings to expand its use and reproduce the bug to be addressed * add docs for setting runtime freq for Example wrapper * prevent crash checking an init or valid time that is a wildcard * clean up function docs * Per #2890, add more checks for skipping times so wrappers don't incorrectly report an error if an init/valid time is skipped. Create function to handle skip checks, logging run time, and adding instance/custom to templates to reduce duplicate code. Change skip logging from DEBUG to INFO so it is more clear what is being skipped. Rename run_at_time function to be more clear of its purpose * update metplus_dev environment to include the documentation requirements * Per #2890, added unit tests for example wrapper to increase code coverage on new code. Improved error reporting if incorrect time configs are set
… removing data from original location as well
…roups that had differences in the last non-ref branch run. runs all use case groups if there is no previous non-ref run to handle first main_vX.Y-ref run before main_vX.Y has been created. also supports workflow dispatch to force update of all truth data as needed
…the run has the information we need
…ow handled in update_truth_data.yml
…ad of actually updating the truth data on DockerHub
* Enhanced AMDAR PBLH METplus use case * Fixes title underline length error. * Adds missing .csv for airport filename in the conf file. * Reverts addition of .csv suffix since the Python embedding script adds it, and adjusts Python embedding script locations to use PARM_BASE instead of INPUT_BASE. * Changes the NumPy where statement to use the string nan instead of the NumPy nan constant to match the data type of the tailNumber column at this point. The string nan is used also on the subsequent line for valid_tails. --------- Co-authored-by: Daniel Adriaansen <dadriaan@ucar.edu>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
pbl:0 use case produces new output files and values.
Created by @DanielAdriaansen