NOAA-EMC / EVS

12 stars 24 forks source link

feature/EVS-GLWU_LC_buoy #569

Closed SamiraArdani-NOAA closed 5 days ago

SamiraArdani-NOAA commented 6 days ago

Note to developers: You must use this PR template!

Description of Changes

This change was added to include NDBC buoy no. 45178 for Lake Champlain region to EVS-GLWU.

Please include a summary of the changes and the related GitHub issue(s). Please also include relevant motivation and context.

There is no GitHub issue. The only motivation for this change is to include Lake Champlain verification to EVS-GLWU.

Developer Questions and Checklist

Would be good if to have the changes earlier that later with the goal to store daily buoy data for Lake Champlain region before freezing time.

Testing Instructions

Please include testing instructions for the PR assignee. Include all relevant input datasets needed to run the tests.

To test this PR, you need to run prep step. 1- Clone my fork in your local and checkout branch feature/EVS-GLWU_LC_buoy 2- ln -s fix directory. 3- Replace $USER with emc.vpppg in driver scripts. 4- Run prep step: $EVS/dev/drivers/scripts/prep/glwu/jevs_glwu_wave_grid2obs_prep.sh.

malloryprow commented 5 days ago

jevs_glwu_wave_grid2obs_prep.sh

Log File: /lfs/h2/emc/vpppg/noscrub/mallory.row/verification/EVS_PRs/pr569/EVS/dev/drivers/scripts/prep/glwu/jevs_glwu_wave_grid2obs_prep.o156487059 COMOUT: /lfs/h2/emc/vpppg/noscrub/mallory.row/verification/EVS_PRs/pr569/evs/v2.0/prep/glwu/wave[.20240926] DATA: /lfs/h2/emc/stmp/mallory.row/evs_test/prod/tmp/jevs_glwu_wave_grid2obs_prep.156487059.cbqs01

SamiraArdani-NOAA commented 5 days ago

@malloryprow, The .o file is free from ERRORs and WARNINGs. Also, The NDBC buoy no. 45178 was added which is the objective of this PR. The output results all look good.

malloryprow commented 5 days ago

Thanks Samira! I'll get this into the beta5 parallel!