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

GHRSST bug fix ... an more #717

Merged
merged 12 commits into from
Nov 14, 2023
Merged

Conversation

guillaumevernieres
Copy link
Contributor

Quite a few bug fixes, but the sstdtime is still not correct.

I added a feature in the testing so our ctest for GHRSST does more than just checking if things run without crashing! The GHRSST application outputs a few basic stats from the ioda variables:

Reading ghrsst_sst_mb_202107010000.nc4
obsVal:
    Min: 276.708
    Max: 276.9
    Sum: 4982.63
obsError:
    Min: 0.32
    Max: 0.32
    Sum: 5.76
...

See file below.
Still not enough but better than doing nothing!

rmclaren
rmclaren previously approved these changes Nov 9, 2023
@apchoiCMD
Copy link
Collaborator

Thanks @guillaumevernieres I will test it on hpc

Copy link
Collaborator

@ShastriPaturi ShastriPaturi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ctest passes

@guillaumevernieres
Copy link
Contributor Author

Thanks @guillaumevernieres I will test it on hpc

Can I assume you approved because you tested on hpc @apchoiCMD ?

@apchoiCMD
Copy link
Collaborator

Yes, it was tested and all are passed. @guillaumevernieres sorry for the late reply!

@guillaumevernieres guillaumevernieres added hera-GW-RT Queue for automated testing with global-workflow on Hera orion-GW-RT Queue for automated testing with global-workflow on Orion labels Nov 13, 2023
@emcbot emcbot added the hera-GW-RT-Running Automated testing with global-workflow running on Hera label Nov 13, 2023
@emcbot emcbot added orion-GW-RT-Running Automated testing with global-workflow running on Orion and removed hera-GW-RT Queue for automated testing with global-workflow on Hera orion-GW-RT Queue for automated testing with global-workflow on Orion labels Nov 13, 2023
@emcbot
Copy link

emcbot commented Nov 14, 2023

Automated Global-Workflow GDASApp Testing Results:
Machine: hera

Start: Mon Nov 13 23:04:23 UTC 2023 on hfe07
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Mon Nov 13 23:51:27 UTC 2023
---------------------------------------------------
Tests:                                  *Failed*
Tests: Failed at Tue Nov 14 00:15:23 UTC 2023
Tests: 81% tests passed, 10 tests failed out of 53
	1584 - test_gdasapp_convert_bufr_temp_dbuoy (Failed)
	1585 - test_gdasapp_convert_bufr_salt_dbuoy (Failed)
	1586 - test_gdasapp_convert_bufr_temp_mbuoyb (Failed)
	1587 - test_gdasapp_convert_bufr_salt_mbuoyb (Failed)
	1588 - test_gdasapp_convert_bufr_tesacprof (Failed)
	1589 - test_gdasapp_convert_bufr_trkobprof (Failed)
	1590 - test_gdasapp_convert_bufr_sfcships (Failed)
	1591 - test_gdasapp_convert_bufr_sfcshipsu (Failed)
	1612 - test_gdasapp_convert_bufr_adpsfc_snow (Failed)
Tests: see output at /scratch1/NCEPDEV/da/Cory.R.Martin/CI/GDASApp/workflow/PR/717/global-workflow/sorc/gdas.cd/build/log.ctest

@emcbot emcbot added hera-GW-RT-Failed Automated testing with global-workflow failed on Hera and removed hera-GW-RT-Running Automated testing with global-workflow running on Hera labels Nov 14, 2023
@emcbot
Copy link

emcbot commented Nov 14, 2023

Automated Global-Workflow GDASApp Testing Results:
Machine: orion

Start: Mon Nov 13 17:02:04 CST 2023 on Orion-login-1.HPC.MsState.Edu
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Mon Nov 13 17:52:43 CST 2023
---------------------------------------------------
Tests:                                  *Failed*
Tests: Failed at Mon Nov 13 18:31:22 CST 2023
Tests: 81% tests passed, 10 tests failed out of 53
	1584 - test_gdasapp_convert_bufr_temp_dbuoy (Failed)
	1585 - test_gdasapp_convert_bufr_salt_dbuoy (Failed)
	1586 - test_gdasapp_convert_bufr_temp_mbuoyb (Failed)
	1587 - test_gdasapp_convert_bufr_salt_mbuoyb (Failed)
	1588 - test_gdasapp_convert_bufr_tesacprof (Failed)
	1589 - test_gdasapp_convert_bufr_trkobprof (Failed)
	1590 - test_gdasapp_convert_bufr_sfcships (Failed)
	1591 - test_gdasapp_convert_bufr_sfcshipsu (Failed)
	1612 - test_gdasapp_convert_bufr_adpsfc_snow (Failed)
Tests: see output at /work2/noaa/stmp/cmartin/CI/GDASApp/workflow/PR/717/global-workflow/sorc/gdas.cd/build/log.ctest

@emcbot emcbot added orion-GW-RT-Failed Automated testing with global-workflow failed on Orion and removed orion-GW-RT-Running Automated testing with global-workflow running on Orion labels Nov 14, 2023
@guillaumevernieres
Copy link
Contributor Author

I'll merge later today (unless somebody objects) since the failure are unrelated to the work done for this PR. @ShastriPaturi , could you have a look at the failing tests?

@guillaumevernieres guillaumevernieres merged commit 03fb9d5 into develop Nov 14, 2023
17 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/ghrsst-more-options branch November 17, 2023 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hera-GW-RT-Failed Automated testing with global-workflow failed on Hera orion-GW-RT-Failed Automated testing with global-workflow failed on Orion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants