-
Notifications
You must be signed in to change notification settings - Fork 31
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
Update for ATMS for End-to-End Test #757
Update for ATMS for End-to-End Test #757
Conversation
End-to-End (using ATMS NPP channel 2 as an example)Comparison QC between UFO and GSI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran 2021080100 gdas cycle from prepatmiodaobs through atmanlfinal. All jobs ran to completion. atms_n20 and atms_npp data processed. Increments look reasonable.
0: Increment print | number of fields = 8 | cube sphere face size: C768
0: eastward_wind | Min:-4.611234e-09 Max:+4.406635e-09 RMS:+6.948029e-12
0: northward_wind | Min:-5.030879e-09 Max:+5.978815e-09 RMS:+7.534733e-12
0: air_temperature | Min:-3.008589e-07 Max:+2.911516e-07 RMS:+1.419730e-10
0: surface_pressure | Min:+0.000000e+00 Max:+0.000000e+00 RMS:+0.000000e+00
0: specific_humidity | Min:-3.916608e-04 Max:+2.488543e-03 RMS:+7.406468e-07
0: cloud_liquid_ice | Min:+0.000000e+00 Max:+1.618770e-20 RMS:+1.293217e-23
0: cloud_liquid_water | Min:+0.000000e+00 Max:+1.474788e-19 RMS:+2.167418e-22
0: ozone_mass_mixing_ratio | Min:+0.000000e+00 Max:+0.000000e+00 RMS:+0.000000e+00
Final diagnostic tarball contains atms diagnostic files.
Orion-login-2:/work2/noaa/da/rtreadon/gdas-validation/comrot/gdas_eval_satwind_JEDI/gdas.20210801/00/analysis/atmos$ tar -tvf gdas.t00z.atmstat
-rw-r--r-- rtreadon/stmp 6888137 2023-11-20 20:17 diag_satwind_ahi_h8_2021080100.nc4.gz
-rw-r--r-- rtreadon/stmp 5797597 2023-11-20 20:17 diag_ascatw_ascat_metop-b_2021080100.nc.gz
-rw-r--r-- rtreadon/stmp 36257317 2023-11-20 20:17 diag_gnssro_2021080100.nc4.gz
-rw-r--r-- rtreadon/stmp 716249805 2023-11-20 20:19 diag_atms_npp_2021080100.nc.gz
-rw-r--r-- rtreadon/stmp 52721306 2023-11-20 20:19 diag_satwind_abi_goes-16_2021080100.nc.gz
-rw-r--r-- rtreadon/stmp 95641266 2023-11-20 20:20 diag_satwind_abi_goes-17_2021080100.nc.gz
-rw-r--r-- rtreadon/stmp 4746726 2023-11-20 20:20 diag_ascatw_ascat_metop-a_2021080100.nc.gz
-rw-r--r-- rtreadon/stmp 728609297 2023-11-20 20:22 diag_atms_n20_2021080100.nc.gz
Aside: as we validate more observation types, the run time for gdasatmanlfinal increases due to gzipping observation diagnostic files. With 8 files we are up to about 5 minutes wall time. We will likely need to parallelize this at a later date.
^[[38;21m2023-11-20 20:17:19,097 - INFO - atm_analysis: Compressing 8 diag files to /work2/noaa/da/rtreadon/gdas-validation/comrot/gdas_eval_satwind_JEDI/gdas.20210801/00//analysis/atmos/gdas.t00z.atmstat.gz^[[0m
^[[38;5;39m2023-11-20 20:17:19,097 - DEBUG - atm_analysis: Gzipping 8 diag files^[[0m
^[[38;5;39m2023-11-20 20:22:07,498 - DEBUG - atm_analysis: Creating tar file /work2/noaa/da/rtreadon/gdas-validation/comrot/gdas_eval_satwind_JEDI/gdas.20210801/00//analysis/atmos/gdas.t00z.atmstat with 8 gzipped diag files^[[0m
^[[38;21m2023-11-20 20:22:10,568 - INFO - atm_analysis: Copying /work/noaa/stmp/rtreadon/RUNDIRS/gdas_eval_satwind_JEDI/gdasatmanl_00/gdas.t00z.atmvar.yaml to /work2/noaa/da/rtreadon/gdas-validation/comrot/gdas_eval_satwind_JEDI/gdas.20210801/00//analysis/atmos^[[0m
^[[38;5;39m2023-11-20 20:22:10,569 - DEBUG - atm_analysis: Copying /work/noaa/stmp/rtreadon/RUNDIRS/gdas_eval_satwind_JEDI/gdasatmanl_00/gdas.t00z.atmvar.yaml to /work2/noaa/da/rtreadon/gdas-validation/comrot/gdas_eval_satwind_JEDI/gdas.20210801/00//analysis/atmos/gdas.t00z.atmvar.yaml^[[0m
Approve.
@@ -2,12 +2,16 @@ observers: | |||
##- !INC ${OBS_YAML_DIR}/amsua_n19.yaml | |||
##- !INC ${OBS_YAML_DIR}/sondes_prepbufr.yaml | |||
##- !INC ${OBS_YAML_DIR}/atms_n20.yaml | |||
##- !INC ${OBS_YAML_DIR}/atms_npp.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do we want gdas-validation developers to update gdas_prototype_3d.yaml
as they merge their gdas-validation work into feature/gdas-validation
?
- Add new observation types to the file but keep them commented out (inactive) ... as is done this PR Update for ATMS for End-to-End Test #757, or
- Add new observations types to the file in active mode (don't comment them out) ... as is done in PR Update GNSSRO assimilation to end-to-end GDASApp validation #753.
- don't change
gdas_prototype_3d.yaml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is my two cents:
We can just leave satwind_g16 active as default and make the rest of the data type inactive.
The newly added data type will be inactive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me. As long as we all follow the same approach, we won't have any conflicts with gdas_prototype_3d.yaml
. We can turn on all or a subset of the validated data once the sprint concludes.
@xincjin-NOAA and I will work on AMSU-A next... |
The ATMS test data for UFO evaluation has been updated on both Orion and Hera. |
The |
Resolved! |
In [ previous PR](#757) for ATMS, the end-to-end testing was done without data thinning. And the excessive QC filtering in clear-sky areas, compared to GSI, was found. (See [plots](#757 (comment))) This PR includes two fixes for ATMS in the end-to-end testing and one new feature: 1. fix the excess QC filtering in clear-sky areas. ---> This has a paring [UFO PR #3122](JCSDA-internal/ufo#3122) 2. add data thinning 3. add the diagnostic flags (QC) --- **This can [reproduce QC flags](#768 (comment)) from GSI in UFO** **To test updates in this PR, please check out the UFO branch: [feature/satrad](https://github.com/JCSDA-internal/ufo/tree/feature/satrad) from JCSDA-internal in gdas-validation. This branch consolidates all proposed code changes to UFO for gdas validation: UFO PR #JCSDA-internal/ufo#3122 UFO PR #JCSDA-internal/ufo#3121 UFO PR #JCSDA-internal/ufo#3094 --------- Co-authored-by: Cory Martin <cory.r.martin@noaa.gov>
List of items to be added in this PR:
BUFR Converter: Add YAML for converting ATMS BUFR to IODA
Testing YAML: re-evaluation results due to update CRTM from v2.3.0 to v2.4.1-jedi.1
Config YAML:
Paring PR in JCSDA UFO repository
Notes: