JCSDA-internal / ioda-converters

Various converters for getting obs data in and out of IODA
8 stars 2 forks source link

Update cris to cris-fsr in NCAR converter #1461

Closed ashley314 closed 5 months ago

ashley314 commented 5 months ago

Description

Updating cris to cris-fsr for the NCAR's obs2ioda converter.

Issue(s) addressed

Addresses https://github.com/JCSDA-internal/skylab/issues/327

Dependencies

List the other PRs that this PR is dependent on:

Impact

Changes the observation ingest suite, so we will test with that CI.

Checklist

BenjaminRuston commented 5 months ago

ok @ashley314 this is essentially hardcoded to the cris-fsr

it is a hard date crossover where NOAA switched processing, unfortunate as the sensor ID seems to be the same case (620); sensor = 'cris-fsr'

ashley314 commented 5 months ago

ok @ashley314 this is essentially hardcoded to the cris-fsr

it is a hard date crossover where NOAA switched processing, unfortunate as the sensor ID seems to be the same case (620); sensor = 'cris-fsr'

@BenjaminRuston When you say "sensor ID" are you referring to case (620)? I did not know how to check that so what I have here was a guess and it seemed to work

BenjaminRuston commented 5 months ago

@ashley314 to be thorough we can get a cris file before the change to cris-fsr and see if it to is caught by the case(620)

if it has a unique identifier that would be great, but I'm guessing they may very well be the same

ashley314 commented 5 months ago

@BenjaminRuston I was able to convert a CrIS file from 2020-01-01 using this converter. But that file seems to contain both the old CrIS and CrIS-fsr. For our skylab purposes, I don't think files older than that will need to be converted

PatNichols commented 5 months ago

@ashley314 Do we have to merge this with the other PRs or is it independent?

ashley314 commented 5 months ago

@PatNichols It should be merged with the other PRs. Thanks for checking!

smaticka commented 5 months ago

When I ran the ingest-observations.yaml after pulling this branch on iodaconv and in ewok and then running make, it completes the ingest experiment. but when I did the same thing, but also pull the branch for skylab, I get an error. I'll write it on the skylab PR.

ashley314 commented 5 months ago

Thanks @smaticka, I followed up in that PR. There is a line in the EWOK PR and you shouldn't be getting that error. Can you check your EWOK repo and do agit pull to make sure the branch is up to date and matches what is in feature/obsingest_cris-fsr?

cmgas commented 5 months ago

If somebody can merge this I will merge the other two!

climbfuji commented 5 months ago

If somebody can merge this I will merge the other two!

Ready whenever you are

cmgas commented 5 months ago

Merged!