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 link_fixdirs.sh for new data directories #672

Merged

Conversation

GeorgeGayno-NOAA
Copy link
Collaborator

@GeorgeGayno-NOAA GeorgeGayno-NOAA commented Jul 20, 2022

DESCRIPTION OF CHANGES:

Update link_fixdirs.sh for new data directories.

  • Point to fix_NEW instead of fix.
  • Add link to fix_fv3_fracoro.
  • Remove link to old directory fix_fv3.

TESTS CONDUCTED:

See below.

DEPENDENCIES:

None.

DOCUMENTATION:

N/A

ISSUE:

Fixes #659.

@GeorgeGayno-NOAA GeorgeGayno-NOAA self-assigned this Jul 20, 2022
directory location on wcoss2 (not Kate's directory).

Fixes ufs-community#659.
@GeorgeGayno-NOAA GeorgeGayno-NOAA marked this pull request as ready for review July 21, 2022 19:48
@GeorgeGayno-NOAA
Copy link
Collaborator Author

The branch at 3d3bb0b was compiled on Hera, Orion, Jet, and WCOSS2. The ./fix/link_fixdirs.sh script was invoked to set the 'fix' directory links. All regression tests passed.

@GeorgeGayno-NOAA
Copy link
Collaborator Author

@KateFriedman-NOAA There are still folks who use the old directories. Will they remain?

@KateFriedman-NOAA
Copy link
Collaborator

@KateFriedman-NOAA There are still folks who use the old directories. Will they remain?

I was hoping to remove the current "fix" folder (older version) to save space. Do you happen to know who is using it? We could potentially keep "fix" with a new name and move folks to it as needed....but really it's a very old fix copy now and folks should be moving away from it. I don't want to impact existing experiments though...might need to set a deadline or something.

Was looking to promote "fix_NEW" to "fix" and implement a new versioning within very soon. That versioning would allow for backwards compatibility and avoid the current issue in the future.

@GeorgeGayno-NOAA
Copy link
Collaborator Author

@KateFriedman-NOAA There are still folks who use the old directories. Will they remain?

I was hoping to remove the current "fix" folder (older version) to save space. Do you happen to know who is using it? We could potentially keep "fix" with a new name and move folks to it as needed....but really it's a very old fix copy now and folks should be moving away from it. I don't want to impact existing experiments though...might need to set a deadline or something.

Was looking to promote "fix_NEW" to "fix" and implement a new versioning within very soon. That versioning would allow for backwards compatibility and avoid the current issue in the future.

@JeffBeck-NOAA had some concerns.

@JeffBeck-NOAA
Copy link
Collaborator

JeffBeck-NOAA commented Jul 27, 2022

Thanks, @GeorgeGayno-NOAA! Hi @KateFriedman-NOAA! My concern was just with the paths. The SRW App uses the old paths to source fixed files, so if the new path would use "fix_NEW", we'd just have to update that in our workflow. If "fix_NEW" were to become "fix", then it wouldn't be an issue. Thanks!

@KateFriedman-NOAA
Copy link
Collaborator

@JeffBeck-NOAA I did not know that the SRW app was using fix, this is great to know, thank you! Ok...sounds like I can proceed with the planned promotion of fix_NEW to fix in the near future. Correct me if I misunderstood. I plan to make a symlink from fix_NEW to fix when that happens to catch folks who still point to fix_NEW. My main concern in this promotion would be if changing the underlying fix files would hurt the systems using it (science-wise). If it's just a path issue (e.g. path suddenly missing) then I'll cover that with symlinks. How does that sound? Any remaining concerns? Thanks!

@GeorgeGayno-NOAA This PR is changing the FIX_DIR point from fix to fix_NEW. If that's to get new files in fix_NEW that don't exist in the current fix set then proceed. If not, you may want to back those path updates out and know that the promotion of fix_NEW to fix will cover that. I leave that up to you.

@JeffBeck-NOAA
Copy link
Collaborator

JeffBeck-NOAA commented Jul 27, 2022

@KateFriedman-NOAA, if there is a symlink from "fix" to "fix_NEW" that should resolve any concerns we have with the SRW App. Then, once you promote "fix_NEW" to "fix", the symlink can be removed and we're back to just a single "fix" directory. That would work! However, if file names changed within the fixed directory, we'll need to update the workflow with the new names. Do you know of any modifications to the fixed files that we might need to be aware of? If it changes things scientifically, say through using higher-resolution fixed data, that'd be great to know!

@KateFriedman-NOAA
Copy link
Collaborator

@JeffBeck-NOAA The fix_NEW set is the fix set plus updates since late last year. The fix set hasn't been updated since then. Never meant for fix_NEW to stay un-promoted this long, lol!

FYI, my plans (if they go as I hope) would be to delete fix, rename fix_NEW to fix, and then symlink fix_NEW to fix. It would look like this afterwards:

fix
fix_NEW -> fix

I just did a diff between fix and fix_NEW to show general diffs between the two. I have put the diff output onto Hera here:
/scratch1/NCEPDEV/global/glopara/diff_fix_fix_NEW.out

Please take a look at the diffs to see if files that the SRW app use are impacted. For the most part, the differences are adding entire new fix file subsets to support GFSv17 into fix_NEW but some existing files were also updated. There were also a few renames (e.g. fix_wave_gfs -> fix_wave).

@JeffBeck-NOAA
Copy link
Collaborator

JeffBeck-NOAA commented Jul 27, 2022

@KateFriedman-NOAA, that all sounds good and shouldn't disrupt anything with the SRW App. Based on your difference file, most things impacted are in "fix_fv3_fracoro", which we don't use. I do see some files were moved to fix_orog and a few files in fix_sfc_climo and fix_am may have been added for vegetation type and rrtmg, respectively. It's possible this will impact things, so I'll just need to run a test case with the App once the change takes place and modify accordingly. Thanks again for the info!

@GeorgeGayno-NOAA
Copy link
Collaborator Author

@JeffBeck-NOAA I did not know that the SRW app was using fix, this is great to know, thank you! Ok...sounds like I can proceed with the planned promotion of fix_NEW to fix in the near future. Correct me if I misunderstood. I plan to make a symlink from fix_NEW to fix when that happens to catch folks who still point to fix_NEW. My main concern in this promotion would be if changing the underlying fix files would hurt the systems using it (science-wise). If it's just a path issue (e.g. path suddenly missing) then I'll cover that with symlinks. How does that sound? Any remaining concerns? Thanks!

@GeorgeGayno-NOAA This PR is changing the FIX_DIR point from fix to fix_NEW. If that's to get new files in fix_NEW that don't exist in the current fix set then proceed. If not, you may want to back those path updates out and know that the promotion of fix_NEW to fix will cover that. I leave that up to you.

@KateFriedman-NOAA I was asked to add a link to fix_fv3_fracoro. Under fix_NEW, there are files that do not exist under fix. And according to @yangfanglin, these fix_NEW files were created with the latest MODIS data. (NOAA-EMC/global-workflow#863 (comment))

@KateFriedman-NOAA
Copy link
Collaborator

It's possible this will impact things, so I'll just need to run a test case with the App once the change takes place and modify accordingly.

@JeffBeck-NOAA Okie dokie...since my intention is to delete the current fix, would it be possible for you to run a test with the SRW app and point it fix_NEW before I make the change? Likely not making the fix_NEW to fix promotion for another week or so.

Note, more info, I keep snapshots of each fix set on HPSS, so the current fix is also on HPSS here:
/5year/NCEPDEV/emc-global/emc.glopara/fv3fix/fix_20211007.tar

I'll likely make a final tarball of fix before I do anything with it, just to be safe.

@KateFriedman-NOAA
Copy link
Collaborator

KateFriedman-NOAA commented Jul 28, 2022

I was asked to add a link to fix_fv3_fracoro. Under fix_NEW, there are files that do not exist under fix. And according to @yangfanglin, these fix_NEW files were created with the latest MODIS data. (NOAA-EMC/global-workflow#863 (comment))

@GeorgeGayno-NOAA Okie dokie...sounds like you should proceed with this change then. After I have completed the promotion of fix_NEW to fix you'll want to change the paths again to remove "_NEW", although a symlink from fix_NEW -> fix will keep things from breaking. Then we'll be sticking with fix moving forward. Walter and I are working on a new versioning construct...I'll send details on that to component managers soon, it should be seamless at the start.

@GeorgeGayno-NOAA GeorgeGayno-NOAA merged commit a33c98a into ufs-community:develop Aug 2, 2022
@GeorgeGayno-NOAA GeorgeGayno-NOAA deleted the feature/new_fix branch August 2, 2022 15:08
AnnetteGibbs-NOAA added a commit to AnnetteGibbs-NOAA/UFS_UTILS that referenced this pull request Jul 3, 2023
* origin/develop:
  Update ccpp submodule (ufs-community#827)
  Change length of character variable "mosaic_name" in "make_solo_mosaic.c" (ufs-community#824)
  Update GDAS INIT utility for GFS COM reorganization (ufs-community#820)
  Add backup calculation for orography and mask generation (ufs-community#713)
  Improve repository build script (ufs-community#819)
  Use latest wgrib2 executable on Jet. (ufs-community#816)
  chgres_cube - Minor update for high-resolution grids . (ufs-community#814)
  Update PR template (ufs-community#813)
  global_cycle - Add soil moisture nudging for NoahMP option (ufs-community#809)
  Release v1.10.0 (ufs-community#807)
  Run sfc_climo_gen utility on Hera (ufs-community#791)
  Fix bug in chgres_cube subroutine search_many (ufs-community#808)
  sfc_climo_gen - Output fractions of each vegetation/soil type category (ufs-community#748)
  Replace w3nco library with w3emc (ufs-community#802)
  Fix Debug compilation (ufs-community#800)
  Update WCOSS2 build module to use new ESMF v8.4.1 library. (ufs-community#799)
  Update chgres_cube documentation for duplicate grib2 records (ufs-community#795)
  Use latest Cray modules on WCOSS2 (ufs-community#792)
  Remove checksum attribute from global_cycle restart files (ufs-community#794)
  Remove goto statements from chgres_cube (ufs-community#775)
  FVCOM_TOOLS - Remove dependency on module_nwp_base.f90 (ufs-community#790)
  Remove "gcovr" step from the 'developer' workflow (ufs-community#785)
  Update build module for Cheyenne (ufs-community#783)
  GDAS Init utility - consolidate the copy of coldstart files (ufs-community#773)
  Remove support for ODIN machine (ufs-community#782)
  Update GDAS INIT utility to optionally use fracoro data (ufs-community#741)
  Option to build only application specific utilities (e.g. GFS) (ufs-community#777)
  Use combined IP/IP2 library (ufs-community#695)
  Initial updates to global_cycle for Noah-MP land model (ufs-community#774)
  New resolution options for the cpld_gridgen utility (ufs-community#769)
  Use new EPIC-maintained hpc-stack on Jet  (ufs-community#771)
  Update GDAS initialization scripts for the new ENKF directory (ufs-community#764)
  Reduce memory usage in chgres_cube (ufs-community#766)
  Detect duplicate vertical levels in chgres_cube. (ufs-community#767)
  Point to new input orography directory. (ufs-community#758)
  Release version 1.9 (ufs-community#754)
  HAFSv1 grid nesting updates (ufs-community#752)
  Remove compiler warnings from chgres_cube (ufs-community#747)
  chgres_cube: Split input_data module into to 3 separate modules (ufs-community#744)
  More work on CI, checking with different versions of ESMF (ufs-community#742)
  Add processing of soil color to sfc_climo_gen (ufs-community#721)
  more work on CI - added Linux_versions workflow (ufs-community#739)
  Add TEST_FILE_DIR option to CMake build, where test data files can be found instead of using FTP.  (ufs-community#732)
  adding developer workflow (ufs-community#724)
  Update GDAS INIT utility for v16.3. (ufs-community#723)
  OROG_GSL - Remove negative bias in orographic asymmetery (OA) fields (ufs-community#718)
  Incorporate BNU soil type data (ufs-community#717)
  High-resolution MODIS and STATSGO veg/soil data. (ufs-community#703)
  Run WCOSS2 consistency tests under role account (ufs-community#711)
  Fix macOS CI workflow (ufs-community#715)
  Add utility codes to create BNU soil texture data (ufs-community#707)
  Update chgres_cube to output netcdf4 file. (ufs-community#704)
  Incorporate high-res global VIIRS vegetation data. (ufs-community#699)
  Run grid_gen consistency tests in parallel. (ufs-community#697)
  Fix error handling in "link_fixdirs.sh". Add -L to the copy command. (ufs-community#701)
  Update for new fixed data directory structure (ufs-community#688)
  Add WCOSS2 support for tests (ufs-community#693)
  Increase test data pull timeout (ufs-community#692)
  Option to install binaries to any directory. (ufs-community#685)
  Update global_cycle to use the latest CCPP version of sfcsub.F (ufs-community#671)
  Incorporate weight_gen program. (ufs-community#677)
  Add install of jpeg-turbo to macos workflows (ufs-community#684)
  Improve logic in regression test driver script (ufs-community#681)
  Release v1.8 (ufs-community#679)
  Update link_fixdirs.sh for new data directories (ufs-community#672)
  Update default ice climatology in ./ush/global_cycle.sh. (ufs-community#664)
  Remove all references to WCOSS 1 from UFS_UTILS (ufs-community#667)
  Update workflows to use latest macOS and ubuntu (ufs-community#675)
  Update GDAS Initialization utility for recent HPSS tarball name change (ufs-community#666)
  Port UFS_UTILS to WCOSS2 (ufs-community#642)
  Fix chgres_cube to process GEFS GRIB2 data (ufs-community#658)
  Update global_cycle_driver.sh for GFS OPS directory convention  (ufs-community#655)
  Update documentation for SRW App (ufs-community#656)
  New coupled model utility (ufs-community#647)
  Update support for S4 and enable regression testing (ufs-community#654)
  global_cycle - Link to CCPP version of sfcsub.F (ufs-community#636)
  Add processing of new global AFWA snow data to emcsfc_snow2mdl. (ufs-community#648)
  Update build module on Cheyenne (ufs-community#646)
  Move to Intel 2022 on Jet, Hera and Orion (ufs-community#650)
  Host doxygen documentation for multiple releases (ufs-community#644)
  Download unit test data as part of the CMake build (ufs-community#630)
  chgres_cube - Complete removal of wgrib2 library (ufs-community#641)
  Eliminate circular dependency in chgres_cube
  Update workflow files to pull netcdf-c library from GitHub
  chgres_cube - Remove the wgrib2 library from the GRIB2 data read routines.
  Undefined symbols on macOS with Intel compiler (ufs-community#628)
  Update FVCOM code to handle sub-domain restart files using multiple cores. (ufs-community#624)
  chgres_cube - Run routine 'convert_omega' on all tasks. (ufs-community#627)
  Use ESMF 8.2.0 library
  Automate update of consistency test baseline data. (ufs-community#603)
  Update workflow files to use newer versions of ESMF and NCEPLIBS. (ufs-community#617)
  Update build modules to be lua compliant (ufs-community#614)
  Allow FVCOM tools to Update Ice Surface Roughness Length (ufs-community#604)
  Update the requested memory in the Orion chgres_cube consistency test script (ufs-community#611)
  Use copy of grib_util under Jet role account. (ufs-community#608)
  Run consistency tests on Orion using role account (ufs-community#606)
  Run consistency tests on Hera using role account (ufs-community#605)
  Run consistency tests on Jet using role account. (ufs-community#607)
  Update more documentation after move to ufs-community (ufs-community#597)
  fvcom_tools - Add option to process 'cold' or 'warm' restart files (ufs-community#595)
  Update documentation after move to ufs-community (ufs-community#594)
  chgres_cube - Eliminate segmentation fault in input_data.F90 (ufs-community#585)
  Update to language of unit test README to match that in unit test.
  chgres_cube - Simplify surface processing using field bundles (ufs-community#572)
  Add compiler flags for GNU Fortran v10 or newer compilers. (ufs-community#583)
  Move verbose output from example unit test to be commented to streamline test output.
  Updates to test README to add instructions for use of example unit test.
  Add Findwgrib2.cmake (ufs-community#578)
  Added unit test to be used as an instructional example for new users.
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.

Incorporate new fixed files
4 participants