Opened 6 years ago

Closed 5 years ago

#574 closed task (fixed)

Changed sign in EUM data reconstruction from I's and Q's [0.5 d]

Reported by: Ian Culverwell Owned by: Ian Culverwell
Priority: normal Milestone: ROPP9.1 carry over
Component: ropp_io Version: 9.0
Keywords: Cc:

Description

Change history (3)

comment:1 by Ian Culverwell, 5 years ago

Very happy to take Stig's guidance on this. It doesn't cause any problems in the automatic ropp_io tests because although t_eum2ropp.sh tests eum2ropp with the -l cl+rs option, which would cause the reader to pass through this bit of the code, our test dataset ropp_io/dataeum_test.n4 doesn't contain any raw sampling data:

idculv@eld037:> ncks -H -Q -v/quality/rs_data_available ../data/eum_test.n4
/quality/rs_data_available
rs_data_available = 0

and so in fact the modified code is bypassed.

comment:2 by Ian Culverwell, 5 years ago

(We do have some cl data, however, so the test of eum2ropp -l cl+rs is not completely fatuous:

idculv@eld037:> ncks -H -Q -v/quality/cl_data_available ../data/eum_test.n4
/quality/cl_data_available
cl_data_available = 1

)

comment:3 by Ian Culverwell, 5 years ago

Resolution: fixed
Status: newclosed

And we don't currently have any tests of EUM lev 1a data in the ropp_pp module, which is why the change has no effect on that. I cannot remember if there are any tests of EUM level 1a data in the Test Folder - if there are, we will probably have to change them.

The whole question of ROPP test data and tests needs revising. Open #655 on the question.

Meanwhile, commit the change at r5940 and close ticket.

Note: See TracTickets for help on using tickets.