Use spack-stack python on Hercules #1133

CoryMartin-NOAA · 2024-05-28T18:52:35Z

As the title says, this moves away from using my maintained python env entirely for GDASApp and mostly for EVA (just EVA and EMCPy are still hosted by me).

Note, I have not tested, hence why it is still a draft PR

modulefiles/GDAS/hercules.intel.lua

aerorahul

looks good. just one comment.

DavidHuber-NOAA

Looks good. Just one suggestion regarding python.

modulefiles/EVA/hercules.lua

RussTreadon-NOAA · 2024-06-05T13:21:38Z

Install feature/hercules-modules on Hercules. Merge develop into working copy. Build updated copy inside g-w develop at 67b833e. Update working copy of g-w sorc/jcb to current head of jcb develop at f62b9df.

Use and load updated modulefiles/GDAS/hercules.intel.lua. Run test_gdasapp with the following results

96% tests passed, 2 tests failed out of 47

Label Time Summary:
gdas-utils    =  16.14 sec*proc (9 tests)
script        =  16.14 sec*proc (9 tests)

Total Test time (real) = 1648.01 sec

The following tests FAILED:
        1759 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN (Failed)
        1763 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_VRFY (Failed)

A check of JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN.out shows that variable APRUN_OCNANALECEN is not defined.

  File "/work2/noaa/da/rtreadon/git/global-workflow/hercules/sorc/gdas.cd/bundle/gdas/ush/soca/marine_recenter.py", line 182, in run
    exec_cmd_gridgen = Executable(self.config.APRUN_OCNANALECEN)
  File "/work2/noaa/da/python/gdasapp/wxflow/20240528/src/wxflow/attrdict.py", line 79, in __getattr__
    return self.__getitem__(item)
  File "/work2/noaa/da/python/gdasapp/wxflow/20240528/src/wxflow/attrdict.py", line 84, in __missing__
    raise KeyError(name)
KeyError: 'APRUN_OCNANALECEN'

This may reflect an inconsistency between GDASApp and g-w and not a problem with this PR.

A check of JGDAS_GLOBAL_OCEAN_ANALYSIS_VRFY.out reveals a conda error

+ slurm_script[52]: set +u
+ slurm_script[53]: conda activate eva
/var/spool/slurmd/job1408168/slurm_script: line 53: conda: command not found
+ slurm_script[1]: postamble slurm_script 1717592082 127

This error appears related to the PR and may require additional changes to modulefiles/EVA/hercules.lua or the soca test.

RussTreadon-NOAA · 2024-06-05T13:24:52Z

A check of /work2/noaa/da/rtreadon/git/global-workflow/hercules/env shows that APRUN_OCNANALECEN is only defined in ORION.env and HERA.env. We should check HERCULES.env and WCOSS2.env to ensure everything we need for marine DA is defined.

CoryMartin-NOAA · 2024-06-14T19:20:42Z

@RussTreadon-NOAA we should probably combine this with the changes we will need for Orion Rocky9

RussTreadon-NOAA · 2024-06-27T17:22:24Z

Hercules remains down due to system issues. Therefore, clone feature/hercules-modules on Orion inside of working copy of g-w PR #2700. Copy modulefiles/GDAS/hercules.intel.lua to modulefiles/GDAS/orion.intel.lua. Replace

prepend_path("MODULEPATH", '/work/noaa/epic/role-epic/spack-stack/hercules/spack-stack-1.6.0/envs/unified-env/install/modulefiles/Core')

with

prepend_path("MODULEPATH", '/work/noaa/epic/role-epic/spack-stack/orion/spack-stack-1.6.0/envs/unified-env-rocky9/install/modulefiles/Core')`

Build GDASApp and run ctests with the following results

Test project /work/noaa/da/rtreadon/git/global-workflow/rename_atm/sorc/gdas.cd/build
      Start 1489: test_gdasapp_util_coding_norms
 1/48 Test #1489: test_gdasapp_util_coding_norms ........................   Passed    7.80 sec
      Start 1490: test_gdasapp_util_ioda_example
 2/48 Test #1490: test_gdasapp_util_ioda_example ........................   Passed    1.71 sec
      Start 1491: test_gdasapp_util_prepdata
 3/48 Test #1491: test_gdasapp_util_prepdata ............................   Passed    1.34 sec
      Start 1492: test_gdasapp_util_rads2ioda
 4/48 Test #1492: test_gdasapp_util_rads2ioda ...........................   Passed    0.15 sec
      Start 1493: test_gdasapp_util_ghrsst2ioda
 5/48 Test #1493: test_gdasapp_util_ghrsst2ioda .........................   Passed    0.15 sec
      Start 1494: test_gdasapp_util_rtofstmp
 6/48 Test #1494: test_gdasapp_util_rtofstmp ............................   Passed    1.00 sec
      Start 1495: test_gdasapp_util_rtofssal
 7/48 Test #1495: test_gdasapp_util_rtofssal ............................   Passed    0.88 sec
      Start 1496: test_gdasapp_util_smap2ioda
 8/48 Test #1496: test_gdasapp_util_smap2ioda ...........................   Passed    0.16 sec
      Start 1497: test_gdasapp_util_smos2ioda
 9/48 Test #1497: test_gdasapp_util_smos2ioda ...........................   Passed    0.15 sec
      Start 1498: test_gdasapp_util_viirsaod2ioda
10/48 Test #1498: test_gdasapp_util_viirsaod2ioda .......................   Passed    0.15 sec
      Start 1499: test_gdasapp_util_icecamsr2ioda
11/48 Test #1499: test_gdasapp_util_icecamsr2ioda .......................   Passed    0.14 sec
      Start 1834: test_gdasapp_check_python_norms
12/48 Test #1834: test_gdasapp_check_python_norms .......................   Passed    9.67 sec
      Start 1835: test_gdasapp_check_yaml_keys
13/48 Test #1835: test_gdasapp_check_yaml_keys ..........................   Passed    1.13 sec
      Start 1836: test_gdasapp_jedi_increment_to_fv3
14/48 Test #1836: test_gdasapp_jedi_increment_to_fv3 ....................   Passed    5.35 sec
      Start 1837: test_gdasapp_setup_cycled_exp
15/48 Test #1837: test_gdasapp_setup_cycled_exp .........................   Passed    3.75 sec

ctests did not finish before 1 pm EDT Orion downtime. Will rerun tests once Orion returns to service.

RussTreadon-NOAA · 2024-06-28T00:26:17Z

Reran ctests on Orion after it returned to service. Unfortunately, 15 tests failed

69% tests passed, 15 tests failed out of 48

Label Time Summary:
gdas-utils    =  25.46 sec*proc (11 tests)
script        =  25.46 sec*proc (11 tests)

Total Test time (real) = 1164.81 sec

The following tests FAILED:
        1843 - test_gdasapp_soca_JGLOBAL_PREP_OCEAN_OBS (Failed)
        1846 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_RUN (Failed)
        1847 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN (Failed)
        1849 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_CHKPT (Failed)
        1850 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_POST (Failed)
        1852 - test_gdasapp_soca_incr_handler (Failed)
        1856 - test_gdasapp_snow_apply_jediincr (Failed)
        1862 - test_gdasapp_atm_jjob_var_init (Failed)
        1863 - test_gdasapp_atm_jjob_var_run (Failed)
        1864 - test_gdasapp_atm_jjob_var_inc (Failed)
        1865 - test_gdasapp_atm_jjob_var_final (Failed)
        1866 - test_gdasapp_atm_jjob_ens_init (Failed)
        1867 - test_gdasapp_atm_jjob_ens_run (Failed)
        1868 - test_gdasapp_atm_jjob_ens_inc (Failed)
        1869 - test_gdasapp_atm_jjob_ens_final (Failed)

Examine soca and atm job log files. Find AttributeError. For example, JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN.out contains

  File "/work/noaa/da/rtreadon/git/global-workflow/rename_atm/sorc/gdas.cd/bundle/gdas/ush/soca/marine_recenter.py", line 48, in __init__
    PDY = self.task_config['PDY']
AttributeError: 'MarineRecenter' object has no attribute 'task_config'
+ JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN[1]: postamble JGDAS_GLOBAL_OCEAN_ANALYSIS_ECEN 1719531579 1

and atmanlinit-18297168.out contains

 File "/work/noaa/da/rtreadon/git/global-workflow/rename_atm/ush/python/pygfs/task/atm_analysis.py", line 29, in __init__
    super().__init__(config)
  File "/work/noaa/da/rtreadon/git/global-workflow/rename_atm/ush/python/pygfs/task/analysis.py", line 30, in __init__
    self.gdasapp_j2tmpl_dir = os.path.join(self.task_config.PARMgfs, 'gdas')
AttributeError: 'AtmAnalysis' object has no attribute 'task_config'
+ slurm_script[1]: postamble slurm_script 1719531925 1

These suggest an issue with wxflow.

Other jobs failed for different reasons. JGDAS_GLOBAL_OCEAN_ANALYSIS_RUN.out contains

 4: Exception: Assertion failed: spaces_.size() >0 in ObsSpaces, line 84 of /work/noaa/da/rtreadon/git/global-workflow/rename_atm/sorc/gdas.cd/bundle/oops/src/oops/base/ObsSpaces.h
10: Exception: Assertion failed: spaces_.size() >0 in ObsSpaces, line 84 of /work/noaa/da/rtreadon/git/global-workflow/rename_atm/sorc/gdas.cd/bundle/oops/src/oops/base/ObsSpaces.h
 6: Exception: Assertion failed: spaces_.size() >0 in ObsSpaces, line 84 of /work/noaa/da/rtreadon/git/global-workflow/rename_atm/sorc/gdas.cd/bundle/oops/src/oops/base/ObsSpaces.h

modulefiles/EVA/hercules.lua

aerorahul · 2024-08-16T18:04:38Z

Any chance this can be resurrected? Orion is also fully spack-stack compliant, so updating that will also be appreciated.

Co-authored-by: Rahul Mahajan <aerorahul@users.noreply.github.com>

CoryMartin-NOAA · 2024-08-16T19:51:31Z

@aerorahul yes I'll build now fresh copies on both orion and hercules and try to debug/get this working next week

CoryMartin-NOAA · 2024-08-16T20:19:42Z

@aerorahul do you know of any issues building UPP in the workflow on Orion?:

Building for machine orion, compiler intel
Lmod has detected the following error:  Cannot load module "stack-intel/2022.0.2" without these module(s) loaded:
   intel/2022.1.2

While processing the following module(s):
    Module fullname       Module Filename
    ---------------       ---------------
    stack-intel/2022.0.2  /work/noaa/epic/role-epic/spack-stack/orion/spack-stack-1.6.0/envs/unified-env/install/modulefiles/Core/stack-intel/2022.0.2.lua
    orion                 /work2/noaa/da/cmartin/work_aug2024_orion/global-workflow/sorc/ufs_model.fd/FV3/upp/modulefiles/orion.lua

CoryMartin-NOAA · 2024-08-16T20:26:17Z

This is still open NOAA-EMC/global-workflow#2694 so I assume the workflow does not work out of the box on Orion?

aerorahul · 2024-08-16T20:29:28Z

@aerorahul do you know of any issues building UPP in the workflow on Orion?:

Building for machine orion, compiler intel
Lmod has detected the following error:  Cannot load module "stack-intel/2022.0.2" without these module(s) loaded:
   intel/2022.1.2

While processing the following module(s):
    Module fullname       Module Filename
    ---------------       ---------------
    stack-intel/2022.0.2  /work/noaa/epic/role-epic/spack-stack/orion/spack-stack-1.6.0/envs/unified-env/install/modulefiles/Core/stack-intel/2022.0.2.lua
    orion                 /work2/noaa/da/cmartin/work_aug2024_orion/global-workflow/sorc/ufs_model.fd/FV3/upp/modulefiles/orion.lua

Yes. The UPP submodule in the ufs-weahther-model does not have the Rocky 8 updates. The PR has been in progress.

aerorahul · 2024-08-16T20:30:35Z

This is still open NOAA-EMC/global-workflow#2694 so I assume the workflow does not work out of the box on Orion?

Correct.
Cycling with GSI is also 2x slower on Orion.

RussTreadon-NOAA · 2024-09-16T17:02:51Z

@CoryMartin-NOAA , should we roll this PR into the GDASApp upgrade to spack-stack/1.8 (GDASapp issue #1283)?

CoryMartin-NOAA · 2024-09-16T18:21:07Z

@RussTreadon-NOAA yes I think so at this point

CoryMartin-NOAA · 2024-11-01T20:28:24Z

Closing this since it is ancient, let's roll it into the next time we upgrade to a newer spack-stack verison.

update hercules modulefiles

6aa35b4

CoryMartin-NOAA requested review from RussTreadon-NOAA, DavidHuber-NOAA and aerorahul May 28, 2024 18:52

aerorahul reviewed May 28, 2024

View reviewed changes

modulefiles/GDAS/hercules.intel.lua Show resolved Hide resolved

aerorahul previously approved these changes May 28, 2024

View reviewed changes

Merge branch 'develop' into feature/hercules-modules

6caf61d

DavidHuber-NOAA suggested changes May 29, 2024

View reviewed changes

modulefiles/EVA/hercules.lua Outdated Show resolved Hide resolved

Update hercules.lua

59dc33f

CoryMartin-NOAA dismissed aerorahul’s stale review via 59dc33f May 29, 2024 13:41

DavidHuber-NOAA approved these changes May 29, 2024

View reviewed changes

CoryMartin-NOAA marked this pull request as ready for review May 30, 2024 14:27

Merge branch 'develop' into feature/hercules-modules

fb37c4f

RussTreadon-NOAA and others added 2 commits June 18, 2024 14:10

Merge branch 'develop' into feature/hercules-modules

89fb6c5

Merge branch 'develop' into feature/hercules-modules

ebdd411

aerorahul reviewed Aug 16, 2024

View reviewed changes

modulefiles/EVA/hercules.lua Outdated Show resolved Hide resolved

CoryMartin-NOAA and others added 3 commits August 16, 2024 15:10

Update modulefiles/EVA/hercules.lua

62c7485

Co-authored-by: Rahul Mahajan <aerorahul@users.noreply.github.com>

Merge branch 'develop' into feature/hercules-modules

3b56e5b

update eva on orion

1580aa8

Merge branch 'develop' into feature/hercules-modules

4ea41ae

CoryMartin-NOAA closed this Nov 1, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use spack-stack python on Hercules #1133

Use spack-stack python on Hercules #1133

CoryMartin-NOAA commented May 28, 2024

aerorahul left a comment

DavidHuber-NOAA left a comment

RussTreadon-NOAA commented Jun 5, 2024

RussTreadon-NOAA commented Jun 5, 2024

CoryMartin-NOAA commented Jun 14, 2024

RussTreadon-NOAA commented Jun 27, 2024

RussTreadon-NOAA commented Jun 28, 2024

aerorahul commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

aerorahul commented Aug 16, 2024

aerorahul commented Aug 16, 2024

RussTreadon-NOAA commented Sep 16, 2024

CoryMartin-NOAA commented Sep 16, 2024

CoryMartin-NOAA commented Nov 1, 2024

Use spack-stack python on Hercules #1133

Use spack-stack python on Hercules #1133

Conversation

CoryMartin-NOAA commented May 28, 2024

aerorahul left a comment

Choose a reason for hiding this comment

DavidHuber-NOAA left a comment

Choose a reason for hiding this comment

RussTreadon-NOAA commented Jun 5, 2024

RussTreadon-NOAA commented Jun 5, 2024

CoryMartin-NOAA commented Jun 14, 2024

RussTreadon-NOAA commented Jun 27, 2024

RussTreadon-NOAA commented Jun 28, 2024

aerorahul commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

CoryMartin-NOAA commented Aug 16, 2024

aerorahul commented Aug 16, 2024

aerorahul commented Aug 16, 2024

RussTreadon-NOAA commented Sep 16, 2024

CoryMartin-NOAA commented Sep 16, 2024

CoryMartin-NOAA commented Nov 1, 2024