JCSDA-internal / ioda-converters

Various converters for getting obs data in and out of IODA
9 stars 4 forks source link

Adding ctest for the updated TROPICS converter #1358

Closed smaticka closed 1 year ago

smaticka commented 1 year ago

Description

The ioda-converter for TROPICS was updated to account for the new calibration by NASA. In this PR, a ctest is created to test that the converter properly works after building jedi.

Issue(s) addressed

Contributes to #1354

Dependencies

Ioda-converter update PR #1355

Checklist

PatNichols commented 1 year ago

@smaticka The file not found error is gone. Now it is seeing differences between the testrun output file and the "gold" standard file in testoutput. DIFFER : LENGTHS OF RECORDS : Location (567) <> Location (2268) DIFFER : LENGTHS : DIMENSION : Location : 567 <> 2268 DIFFER : VARIABLE : Location : ATTRIBUTE : suggested_chunk_dim : VALUES : 567 <> 2268 DIFFER : VARIABLE : Location : SIZE : 567 <> 2268 DIFFER : VARIABLE : dateTime : SIZE : 567 <> 2268 DIFFER : VARIABLE : latitude : SIZE : 567 <> 2268 DIFFER : VARIABLE : longitude : SIZE : 567 <> 2268 ...more....

smaticka commented 1 year ago

Thanks for the comments everyone. This test passes now! The testoutput file corresponded to a previous ioda-conversion.

It should be ready for review