SDU-C/D-CIDA-2/3-NEXT-TEMPEL1-V1.0
==================================


Result: Not certified; delta-review of this data set after resolution of liens marked [2] below.


Recommendation: 
===============
Archive this data set with liens resolved sufficient to make this a reasonable "raw" (CODMAC 2) 
data set.  These liens are marked with [2] in the list below.  The liens marked [4] should be 
answered by a new, high-level data set.


Liens:
======

o [2] In the dataset.cat file, there is a reference to CIDASYS.TXT which should be CDASIS.ASC.

o [2] In the header.fmt file, provide a meaningful DESCRIPTION for the event time column.

o [2] The target signal and spectrum should be extracted into separate tables.

o [2] Remove the MASS HINT column.


o [4] Add characteristic impact parameters, such as charge yield

o [4] Calibrated mass spectra are not provided.  Is it possible to provide TOF, mass, amplitude and 
      (if possible) identification of mass lines?  Or does this need to be a different data set?  
      These are needed for the isp/ and tempel1/ subdirectory.

o [4] If possible, create spectra that combine measurements from all amplifiers into a single spectrum.

o data/ancillary/keep_t20050611_t20110214.tab
  - For column 1, what checksum algorithm was used? The only description I
    found:
    DESCRIPTION = "Checksum of the keepalive data"
    Many values for this field are negative, which might be wrong.

o data/ancillary/*.tab
  - TABLEs must have at least 1 COLUMN at the top level. If the intent in using
    containers is to allow numbering of columns in all *.fmt to start from 1, one
    workaround is to remove the container around only ^structure =
    cida_header.fmt, i.e. change
    OBJECT               = CONTAINER
      NAME         = CIDA_HEADER
      START_BYTE  
      file://localhost/var/folders/4M/4MFJOTKMHheiKP4QLutEtU++-OA/-Tmp-/com.apple
      .PhotoBooth-T0x1001179c0.tmp.2ZSLlo/4-up%20on%202011-11-12%20at%2019.08.jpg
      = 1
      BYTES        = 250
      REPETITIONS  = 1
      DESCRIPTION  = "CIDA header"
      ^STRUCTURE   = "CIDA_HEADER.FMT"
    END_OBJECT           = CONTAINER
    to
    ^STRUCTURE   = "CIDA_HEADER.FMT"
    (and reset the start byte in ^TABLE)

o catalog/next.cat 
  -  Has gaps in the mission phase table.  These should be explained, preferably
     by documenting the idle interstitial days in the phase table.  Also check the 
     "Spacecraft Operations Type" values in this table; and it would probably be a 
     good idea to proof everything in the table.
  -  One version ahead of the other datasets'.
  -  line 30   "15.February" -> "15 February"
  -  line 89   "200X" -> "2006"
  -  line 122  "CIDA on" -> "CIDA was on"
  -  line 123  "data taken" -> "data were taken"
  -  line 126  "Performed navigation maneuvers" -> "Navigation maneuvers were 
     performed"
  -  line 142  T is used here for the time of encounter, but E was used in 
     navcam.cat. Please standardize this usage. Also, please define
     T (or E).
  -  line 166, remove "." after 15
  -  line 189  "26.March" -> "25 March"
  -  line 192: "earth" -> "Earth"

o catalog/dataset.cat 
  -  line 397  "on on" -> "on"
  -  Has host ID of "SDU", but this is "N/A" in the instrument file.
     This discrepancy needs to be resolved.

o VOLDESC.CAT is missing RECORD_TYPE.

o catalog/reference.cat
  -  BROWNLEEETAL2003 is missing author F. Horz
  -  After first author, initials precede last name, e.g. for
     "BELTONETAL2011":
     REFERENCE_DESC = "Belton, M.J.S., Meech, K.J., Chesley, S., ..." should be
     REFERENCE_DESC = "Belton, M.J.S., K.J. Meech, S. Chesley, ..."
  -  DUXBURYETAL2004 should be DUXBURYETAL2004B
  -  missing references from other datasets' reference.cat (which may be
     desired):
     SEMENOVETAL2004B, ECONOMOUETAL2011, GREENETAL2004, TUZZOLINO1994

o catalog/sdu.cat
  -  line 134  "15.February" -> "15 February"

o aareadme.txt 
  -  line 23  "considered and integral" -> "considered an integral" [in all three
     aareadme.txt files]