Release/1J00_1700

[rsjoyner]

Summary

Updated to newest version of the CART LDD; 1J00_1970
Updated to IM version 1.20.00

[lylehuber]

Should say PDS4_CASSINI_1J00_1700 somewhere?

[jordanpadams]

@lylehuber @rsjoyner missing update to pds4_versions.txt file to include 1J00 version

[jordanpadams]

@lylehuber also looks like there is a bug somewhere: https://github.com/pds-data-dictionaries/ldd-cassini/actions/runs/8378014955/job/22941538520

[jordanpadams]

[jordanpadams]

@lylehuber @rsjoyner If this is a bug in LDDTool, we can open a ticket over there.

[rsjoyner]

Yes. I think this is a bug in LDDTool. I will work with Steve to make sure that I am not doing something way stupid in the LDDTOOL command.

[jordanpadams]

@rsjoyner do you need to include the CART LDD as well since the schematron references it?

[jordanpadams]

@rsjoyner that is the issue... do the CART LDD needs to be added to his repo.

https://pds-data-dictionaries.github.io/getting-started/ldd-repo-overview.html#handling-ldd-dependencies

[jordanpadams]

Let me know if you run into issues

[rsjoyner]

While we are waiting for Steve to reply. See the enclosure for the edited version of the schematron file with the required fix.
Is there a way to use these files in lieu of the "build" ?

CASSINI_LDD_1J00_1700_20240320.zip

[jordanpadams]

@rsjoyner @lylehuber this is not a bug in LDDTool:

lddtool -lpJ ldd-cassini/src/PDS4_CASSINI_IngestLDD.xml ldd-cart/src/PDS4_CART_IngestLDD.xml

produces all valid files.

So if we add that to the dependencies to this repo per these instructions, the automation will work and the files will be accurate

[rsjoyner]

lddtool -lpJ ldd-cassini/src/PDS4_CASSINI_IngestLDD.xml ldd-cart/src/PDS4_CART_IngestLDD.xml

1. I tried a functionally similar command on local copies of the 3 LDDs (e.g., cassini, cart, and geom):

lddtool_1E30_edited -lpJ1 -V 1J00 PDS4_CASSINI_IngestLDD.xml PDS4_CART_IngestLDD.xml PDS4_GEOM_IngestLDD.xml > log-a.txt

lddtool crashed again.

2. I tried again referencing only the 2 LDDs (e.g., cassini and cart) and lddtool crashed again).

I am using:

LDDTool Version: 14.3.0
Built with IM Version: 1.21.0.0
Build Date: 2023-09-18 21:33:49
Configured IM Versions: [1L00, 1K00, 1J00, 1I00, 1H00, 1G00, 1F00, 1E00, 1D00, 1C00, 1B10, 1B00]

[jordanpadams]

@lylehuber @rsjoyner this is fixed and ready for review/merge

[lylehuber]

I hope this works now.

[jordanpadams]

@lylehuber one minor update to remove an unnecessary submodule for ldd-geom.

[jordanpadams]

A fun first corner case of nested dependencies. Had to implement recursion to handle it.

[jordanpadams]

@lylehuber I merged it for you. Once it tags a release here, you can submit a ticket to get it put online.