c-proof / cproofwebsite

Other
0 stars 3 forks source link

Bad delayed mode data #26

Open jklymak opened 1 year ago

jklymak commented 1 year ago
jklymak commented 1 year ago

Actually most of the delayed mode data is not usable on Line P except for dfo-walle652-20191209?

I think we should be more careful about posting clearly broken data.

jklymak commented 1 year ago
jklymak commented 1 year ago

So a few things

  1. delayed mode data does not need such a long filter length for the profile finding. 20 s versus 400 s is fine and gives much better profiles
  2. Some of the delayed mode data was being gridded from the non-delayed mode files in a reprocessing. That lead to a huge number of gaps as the realtime data wasn't sampled as often.
  3. Naming files - the old code would just make dirname/dfo-rosie713-20220531.nc. But @hvdosser had a naming convention of dirname/dfo-rosie713-20220531_delayed.nc. Not sure how that file was bing made? manual copy?
    • Added new optional argument to ncprocess.py:make_gridfiles fnamesuffix. This defaults to empty, but could be fnamesuffix='_delayed' to add that suffix.
    • OTOH, why are these not just stored in a different subdirectory? L0-timeseries->L0-timeseries-delayed seems easier to track? But I didn't want to make that change if a lot of directories were made the other way.
hvdosser commented 1 year ago

I'm working through all the delayed mode data for all missions, and would appreciate input on the following:

I'm currently waiting on James to upload the delayed mode data for the two 2022 Line P missions, and will process it ASAP when it shows up (hopefully today or tomorrow - he's located it and I'm just waiting on the upload).