Menu

#2467 Old LANL CDF kludge is messing up PSP data

nextrelease
open
nobody
None
5
2022-06-29
2022-06-29
No

There's an old LANL CDF kludge, where if a DEPEND_1 has two records and can be interpreted as min and max, then the average is returned. This is causing problems when reading some Parker Solar Probe data, which is a CDF file with just two records.

Discussion

  • Jeremy Faden

    Jeremy Faden - 2022-06-29

    I've put in a check to see that this would result in a valid dataset, checking that the number of records expected isn't two, and this allows the code to do more.

     
  • Jeremy Faden

    Jeremy Faden - 2022-06-29

    Also, QDataSet has been able to represent min,max ranges for a while, and this kludge might be updated to support this.

     
  • Jeremy Faden

    Jeremy Faden - 2022-06-29

    Another issue that's related: Sadie and Cindy were using the "X" and "Y" selections of the CDF editor, without selecting a variable for the first dimension (Z). This should be allowed, since it's explicit. And, the list of parameter names was not the same on the three lists, and I need to remind myself why this is.

    And in the waveform case, the three datasets should be flattened to allow more easy use, and since DEPEND_0 cannot be rank 2 and a BUNDLE is of rank 1 datasets.

     
MongoDB Logo MongoDB