SOHO_0001
=========

- Missing AAREADME.TXT and VOLDESC.CAT
- Mixed case - some directories are uppercase, some lower.


calib/
------

calibinfo.txt
 - INcorrect file name.  This should be called "calinfo.txt".


calib/filter/
-------------

*.lbl
 - The noun form of the verb "transmit" is "transmission", NOT "transmittance".
 - Column 2 ("TRANSMITTANCE") needs a new name and a UNIT specifier.
 - SPACECRAFT_NAME is an obsolete keyword.  Use INSTRUMENT_HOST_NAME instead.
 - Missing the following required keywords:
     TARGET_NAME
     START_TIME
     STOP_TIME
     INSTRUMENT_HOST_NAME

calib/quantum_eff/
------------------

c2_qe.lbl & C3_qe.lbl
 - "MSB_INTEGER" is not a valid data tye for ASCII tables - should be
   "ASCII_INTEGER".
 - Column 2 ("QUANTUM_EFFICIENCET") needs a UNIT specifier.
 - SPACECRAFT_NAME is an obsolete keyword.  Use INSTRUMENT_HOST_NAME instead.
 - Missing the following required keywords:
     TARGET_NAME
     START_TIME
     STOP_TIME
     INSTRUMENT_HOST_NAME


calib/vignetting/
-----------------

*.lbl
 - TARGET_NAME cannot be "UNK".  It must be a real target name.
 - START_TIME cannot be "UNK". It may be "N/A".
 - STOP_TIME should not be "UNK" in this context.  It may be "N/A".
 
vign.txt
 - I'm not comfortable with this information being in a disposable TEXT file.
   It really ought to be a document.


catalog/
--------

soho_ds.cat
 - DATA_SET_NAME must include the version number.
 - DATA_SET_TARGET object mmust be repeated for every unzique TARGET_NAME
   (in the same way DATA_SET_REFERENCE_INFORMATION is repeated).

sohohost.cat
 - RECORD_TYPE does not belong in catalog files.
 - There is no attribution for the descriptive text.

sohoinst.cat
 - RECORD_TYPE does not belong in catalog files.

soho_mis.cat
 - RECORD_TYPE does not belong in catalog files.
 - I like the "Author" subheading at the end of the descriptive information.
   We should consider using this convention elsewhere.
 - REFERENCE_KEY_ID should be "N/A" if there is no reference, not a 
   zero-length string.


data/
-----
*.lbl
 - Some of these non-existent keywords have been submitted for CN approval,
   but I don't recognize SOLAR_NORTH_POLE_CLOCK_ANGLE.  Where is the 
   element definition file for this keyword?
 - How can EXPOSURE_DURATION and STOP_TIME be known but 
   SPACECRAFT_CLOCK_STOP_COUNT isn't?
 - Are all those trailing zeroes on DERIVED_MINIMUM and 
   SAMPLING_PARAMETER_INTERVAL accurate and/or necessary?
 - "ARCSEC" should probably be "ARCSECOND".
 - DERIVED_MINIMUM and DERIVED_MAXIMUM may only be used when there is also
   a SCALING_FACTOR and/or OFFSET.  Neither of these is present in the label.
   So what, exactly, are the DERIVED_MINIMUM/MAXIMUM supposed to be?


document/comet_pos/
-------------------
- Why are these tables not set up as data tables?


document/comet_pos/raw_astrometry/
----------------------------------
 - Why are these tables not set up as data tables?
 - Why are these files named by SOHO serial number and not comet designation,
   which would allow them to be quickly matched to the relevant data?


document/handbook/
------------------
 - Who owns the copyright and where's the acknowledgement?  Do we have
   permission to republish this?


document/occulter_center/
-------------------------
 - XV cannot display these PNG files.  Are they standard?  Can anyone display
   them?


document/spacecraft_info/
-------------------------
 - We do have permission to republish this stuff, don't we?


document/sun_position/
----------------------
 - These file names have mixed case.
 - XV cannot display these PNG files.
 - Do we have permission to republish the text?
 - Why is the SC_body_axis_avg_roll.txt file not a data table?


document/telemetry_stats/
-------------------------
 - Is the list in tm_stat.txt necessary?  Wouldn't it just be easier to note
   the half dozen months in the total range that DON'T have plots?
 - XV could not display these PNG files.