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

Simplified AMV bufr2ioda.py input/output #328

Merged
merged 2 commits into from
Mar 24, 2025

Conversation

HuiLiu-NOAA
Copy link
Collaborator

@HuiLiu-NOAA HuiLiu-NOAA commented Mar 14, 2025

This PR simplifies the input and output parameters with only $CDATE left in the json file.

For the cycle 2024052700, the brief available AMVs information generated by this python bufr2ioda_satwnd tool is listed below:

    Total AMV number over CONUS = 12612

Available variables:
$~~~~~~~~~~~$ dataProvider
$~~~~~~~~~~~$ dateTime
$~~~~~~~~~~~$ latitude, longitude, pressure

$~~~~~~~~~~~$ prepbufrReportType:
$~~~~~~~~~~~$ 245, $~~~~~~~~~~~$ 246, $~~~~~~~~~~~$ 247
$~~~~~~~~~~~$ GOES IR, WV cloud top, WV deep layer
$~~~~~~~~~~~$ 60% $~~~~~~~~~~~$ 30% $~~~~~~~~~~~$ 10% (of total AMVs)

$~~~~~~~~~~~$ satelliteIdentifier (= prepbufr subtype):
$~~~~~~~~~~~$ 270 (GOES 16), $~~~$ 272 (GOES 272)

     qiWithoutForecast,  percentErrorEE, windQualityMarker
     satelliteZenithAngle
     trackingCorrelation
     windCalculationMethod
     channelCentralFrequency
     coefficientOfVariation

$~~~~~~~~~~~$ ObsValue
$~~~~~~~~~~~$ windDirection
$~~~~~~~~~~~$ windEastward
$~~~~~~~~~~~$ windNorthward
$~~~~~~~~~~~$ windSpeed

$~~~~~~~~~~~$ ObsError
$~~~~~~~~~~~$ windEastward
$~~~~~~~~~~~$ windNorthward

ioda_satwnd_dc

@HuiLiu-NOAA HuiLiu-NOAA changed the title simplified bufr2ioda.py input/output Simplified bufr2ioda.py input/output Mar 14, 2025
@HuiLiu-NOAA HuiLiu-NOAA self-assigned this Mar 14, 2025
ShunLiu-NOAA
ShunLiu-NOAA previously approved these changes Mar 17, 2025
@HuiLiu-NOAA HuiLiu-NOAA changed the title Simplified bufr2ioda.py input/output Simplified AMV bufr2ioda.py input/output Mar 17, 2025
Copy link
Contributor

@SamuelDegelia-NOAA SamuelDegelia-NOAA left a comment

Choose a reason for hiding this comment

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

Is there a reason we want to remove the input and output directories options (dump_dir and ioda_dir)? It seems to me that keeping these as arguments passed in by run_bufr2ioda.sh would be easier. But maybe this is already handled in rrfs-workflow?

From Hui:

Yes, the paths are already handled in the workflow, for all obs types in data/obs/.

@ShunLiu-NOAA
Copy link

@keenaneure @hu5970 and @guoqing-noaa Please review this PR if you have a chance. We will merge this one by the end of Friday.

@keenaneure
Copy link
Collaborator

Looks good to me.

@ShunLiu-NOAA ShunLiu-NOAA merged commit 9204bdc into NOAA-EMC:develop Mar 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants