SBN:T.Barnes review comments 2025-10-17

===============================================
NOTE:

===============================================
Datasets Validated:
	urn:nasa:pds:liciacube:luke_tf_calibrated::1.0


===============================================
Comments for:	urn:nasa:pds:liciacube:luke_tf_calibrated::1.0

Should the OLAF software be cited due to its help generating the labels?

Target_name
--> There is a mismatch between what LICIACube team provided as the target and what this collection provides for the same images.  Why are not any of the data products have Dimorphos (urn:nasa:pds:context:target:satellite.65803_didymos.dimorphos) listed?  When should the target be dimorphos vs didymos?  Or should both be listed?

file: 'collection.xml'
--> The Citation_Information.description is very terse.  Please expand on it.  Think of this as the abstract for your data collection.  Include details like alternate calibration, target(s) observed, use of alternate spice kernels, merged triplet images, etc.
--> Please Funding_Acknowledgement information (funding source, year, award, acknowledgement) to the Citation_Information.description or if you update the IM version, you can add a Funding_Acknowledgement class.
--> The target Dimorphos (urn:nasa:pds:context:target:satellite.65803_didymos.dimorphos) should be added.
--> The Reference_List item for the data_summary should be LIDVID not LID.
--> Add to the Reference_List Franham et al. (2025).
--> Add to the Reference_List any of the references in the data_summary.txt file that are required for this data collection (citations as opposed to readings).
--> Please add the Primary_Result_Summary.processing_level "Derived" to account for the merged products.
--> Why are there so many Science_Facets listed, when all the data products only list Spectroscopy?

file: 'data_summary.txt'
--> The blank sky (frame liciacube_luke_l0_1664234361_00008_01.fits) image should be referenced, since it is not found in this collection.  I believe LIDVID reference is "urn:nasa:pds:liciacube:luke_raw:liciacube_luke_l0_1664234361_00008::1.0".
--> Remember to update the lines labeled "**** Update with new NAIF names" since these have been released (but if they haven't please update the text to indicate)
--> Farnham et al. reference DOI should be updated from "XXXXXXXX" to the actual DOI value.  If this is the same as the farnham_dart_boulders.pdf, you could also mention this copy exists.

files: 'data/*/*.xml'
--> The <title> for each file is exactly the same.  Please make them unique.  Recommend adding the product part of the LID.
--> The <title> clearly says it is a merged image for Didymos and Dimorphos, but the Target_Identification only lists Didymos.
--> The collection.xml files lists many Science_Facets that are not found in these files.
--> Please add to the <Science_Facet> the <wavelength_range>.
--> Spice files not listed.

files: 'data/calibrated/*.xml'
--> The product(s) used to calibrate these products are not referenced in the label.  
--> --> blank sky frame (liciacube_luke_l0_1664234361_00008_01.fits, LIDVID urn:nasa:pds:liciacube:luke_raw:liciacube_luke_l0_1664234361_00008::1.0).
--> --> raw image LIDVID
--> --> any others?
--> Highly recommend adding the sb:Calibration_Information class to describe the calibration information.
--> Where does it describe which Color Channel is which color in the Array_3D_Image?  Please add in the description.  The disp:Color_Display_Settings appears to list which channel goes to which.  Is this sufficient?
--> Recommend adding a reference to the paper or document that describes the calibration process (farnham_dart_boulders?).
--> Something odd, the start/stop_time and img:exposure_duration implies a exposure of 0.000 for several images.  I see this in the raw files as well.

files: 'data/merged/*.xml'
--> Please add Source_Product_Internal references to the calibrated products in this collection that were used to generate each file.
--> Recommend adding a reference to the paper or document that describes the merging process (data_summary.txt or farnham_dart_boulders?).

file: 'document/farnham_dart_boulders.xml'
--> Update the title to match that of the paper and mention it is a draft.
--> Update the Citation_Information.description to reflect the current situation of this paper.
--> Should the PDF file be updated with the pre-published version?
--> Please add to the Reference_List the published copy of this paper.