YAMLs and Python script for the AVHRR BUFR#69
Open
PraveenKumar-NOAA wants to merge 3 commits intodevelopfrom
Open
YAMLs and Python script for the AVHRR BUFR#69PraveenKumar-NOAA wants to merge 3 commits intodevelopfrom
PraveenKumar-NOAA wants to merge 3 commits intodevelopfrom
Conversation
PraveenKumar-NOAA
requested review from
emilyhcliu,
nicholasesposito and
rmclaren
Contributor
nicholasesposito
left a comment
nicholasesposito
left a comment
There was a problem hiding this comment.
One line has an error. Just update it and then it is approved.
Contributor
Author
|
@emilyhcliu I have created brightnessTemperature plots for the metop-b and metop-c channels and added to the following issue -- #63. |
rmclaren
previously approved these changes
Aug 29, 2025
nicholasesposito
previously approved these changes
Aug 29, 2025
Contributor
nicholasesposito
left a comment
nicholasesposito
left a comment
There was a problem hiding this comment.
Thanks! Approved
rmclaren
dismissed stale reviews from nicholasesposito and themself
The base branch was changed.
HyundeokChoi-NOAA
added a commit
that referenced
this pull request
Apr 7, 2026
…m Praveen.Kumar (#101) This PR have 2 updates. 1. Enable extraction of redistribution‑restriction metadata by adding restrictionFlag (RSRD) and restrictionExpiration (EXPRSRD) to the BUFR→IODA conversion. This ensures downstream restriction‑filter logic has the required fields available in MetaData. 2. Add modified IODA-Converters that originally from Praveen in PRs (#45, #55, #61, #62, #69). --------- Co-authored-by: hyundeok choi <[email protected]> Co-authored-by: hyundeok choi <[email protected]> Co-authored-by: Cory Martin <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: hyundeok choi <[email protected]> Co-authored-by: hyundeok choi <[email protected]> Co-authored-by: hyundeok choi <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR introduces the following files for the AHVRR BUFR converter:
~/dump/mapping/bufr_avhrr.yaml
~/dump/mapping/bufr_avhrr.py
~/ush/test/config/bufr_bufr4backend_avhrr.yaml
~/ush/test/config/bufr_script4backend_avhrr.yaml
Testing and Validation:
All four workflows (bufr2netcdf, bufr4backend, script2netcdf, and script4backend) were executed and tested using serial execution.
Validation tests were completed for the following variables:
-- satelliteZenithAngle
-- brightnessTemperature ( a comparision is required with GSI or other reference data for each channel specific output IODA files)
The validation was performed by comparing the input BUFR files and the corresponding output IODA files. Additional details can be found in #63.