NOAA-EMC / GDASApp

Global Data Assimilation System Application
GNU Lesser General Public License v2.1
15 stars 31 forks source link

Commenting out more of the marine bufr 2 ioda stuff #1018

Closed guillaumevernieres closed 6 months ago

guillaumevernieres commented 6 months ago

I wasn't thinking when merging Shastri's PR, it needs its g-w companion PR. @AndrewEichmann-NOAA , we'll have to revert this once you sort out things on the g-w side.

emcbot commented 6 months ago

Automated Global-Workflow GDASApp Testing Results: Machine: hera

Start: Wed Apr  3 18:07:05 UTC 2024 on hfe08
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Wed Apr  3 18:58:20 UTC 2024
---------------------------------------------------
Tests:                                  *Failed*
Tests: Failed at Wed Apr  3 19:27:11 UTC 2024
Tests: 94% tests passed, 3 tests failed out of 53
    1763 - test_gdasapp_soca_JGDAS_GLOBAL_OCEAN_ANALYSIS_VRFY (Failed)
    1776 - test_gdasapp_atm_jjob_var_run (Failed)
    1777 - test_gdasapp_atm_jjob_var_final (Failed)
Tests: see output at /scratch1/NCEPDEV/da/Cory.R.Martin/CI/GDASApp/workflow/PR/1018/global-workflow/sorc/gdas.cd/build/log.ctest
emcbot commented 6 months ago

Automated Global-Workflow GDASApp Testing Results: Machine: orion

Start: Wed Apr  3 13:05:27 CDT 2024 on Orion-login-1.HPC.MsState.Edu
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Wed Apr  3 13:59:08 CDT 2024
---------------------------------------------------
Tests:                                  *Failed*
Tests: Failed at Wed Apr  3 14:28:29 CDT 2024
Tests: 96% tests passed, 2 tests failed out of 53
    1777 - test_gdasapp_atm_jjob_var_run (Failed)
    1778 - test_gdasapp_atm_jjob_var_final (Failed)
Tests: see output at /work2/noaa/stmp/cmartin/CI/GDASApp/workflow/PR/1018/global-workflow/sorc/gdas.cd/build/log.ctest
AndrewEichmann-NOAA commented 6 months ago

@guillaumevernieres is it not enough to comment out the in situ obs in the obs yaml?

guillaumevernieres commented 6 months ago

@guillaumevernieres is it not enough to comment out the in situ obs in the obs yaml?

It's not. It's not finding a python module. The fix was in Shastri's g-w PR, which you took over today.