search

NOAA-OWP / evapotranspiration

Other
3 stars 9 forks source link

Required updates for integration workflow #33

Closed madMatchstick closed 7 months ago

madMatchstick commented 7 months ago

Includes two fixes to ngen_integration.yaml workflow

  1. Passes cmake build flag -DNGEN_IS_MAIN_PROJECT:BOOL=ON to NOM lib build
  2. Set's PET submodule initialize build to false. This allows for actual updates in PRs to be tested. Otherwise, the specific master branch commit pinned in ngen's extern/evapotranspiration will overwrite any local code changes copied to the model's working directory.

See relevant discussions here and here.

Testing

  1. All workflows passed in manual action runs. See job details.