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

Update realtime branch, gsl_ufs_rt #75

Merged
merged 2 commits into from
Mar 6, 2025

Conversation

kayeekayee
Copy link
Collaborator

  • Ran 120 hours on Hera: /scratch2/BMC/gsd-fv3-dev/KaYee.Wong/global/2025FebMerge/test/rt_merge_dev_Mar05
  • Verified output with existing real time results.


#KYWexport FORECAST_HOUR jobid
export jobid="${job}.$$"
fhr3_base="10#${FHR3}"

Check notice

Code scanning / shellcheck

Possible misspelling: FHR3 may not be assigned. Did you mean fhr3? Note

Possible misspelling: FHR3 may not be assigned. Did you mean fhr3?
# GFS restart interval in hours
#JKHexport restart_interval_gfs=12
#KYWexport restart_interval_gfs=-1 ## JKH
export restart_interval_gfs=$FHMAX_GFS

Check notice

Code scanning / shellcheck

Prefer putting braces around variable references even when not strictly required. Note

Prefer putting braces around variable references even when not strictly required.
# shellcheck disable=SC2153
export FHMAX=${FHMAX_GFS}
# Convert comma-separated string into bash array
IFS=', ' read -ra segments <<< "${FCST_SEGMENTS}"

Check warning

Code scanning / shellcheck

segments appears unused. Verify use (or export if used externally). Warning

segments appears unused. Verify use (or export if used externally).
IFS=', ' read -ra segments <<< "${FCST_SEGMENTS}"
# Determine MIN and MAX based on the forecast segment
export FHMIN=$FHMIN_GFS
export FHMAX=$FHMAX_GFS

Check notice

Code scanning / shellcheck

Possible misspelling: FHMAX_GFS may not be assigned. Did you mean FHMAX_HF? Note

Possible misspelling: FHMAX_GFS may not be assigned. Did you mean FHMAX_HF?
@jderrico-noaa jderrico-noaa merged commit f9b655a into NOAA-GSL:gsl_ufs_rt Mar 6, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants