Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve error handling for LDDTool processing of CTLI IngestLDD file #91

Closed
jshughes opened this issue Nov 27, 2019 · 6 comments
Closed
Labels
bug Something isn't working

Comments

@jshughes
Copy link
Collaborator

LDDTool aborts while processing CTLI IngestLDD file. The reported error is Exception in thread "main" java.lang.ClassCastException: DOMAttr cannot be cast to DOMClass. Changed test for class name from MOF "AttrDefn" to DOM "DOMAttr".

jshughes416 pushed a commit that referenced this issue Nov 27, 2019
LDDTool aborts while processing CTLI IngestLDD file. The reported error is Exception in thread "main" java.lang.ClassCastException: DOMAttr cannot be cast to DOMClass. Changed test for class name from MOF "AttrDefn" to DOM "DOMAttr".
jshughes added a commit that referenced this issue Nov 27, 2019
…CTLI-IngestLDD-Abort

Issue #91 Fix LDDTOOL CTLI IngestLDD Abort
@jordanpadams jordanpadams changed the title Fix LDDTOOL CTLI IngestLDD Abort Improve error handling for LDDTool processing of CTLI IngestLDD file Dec 2, 2019
@jordanpadams jordanpadams added the bug Something isn't working label Dec 2, 2019
@rchenatjpl
Copy link

What is CTLI? Can you send me or attach such an Ingest_LDD file? Thanks

@jshughes
Copy link
Collaborator Author

CLTI - Context Type List - Instrument - see https://pds.jpl.nasa.gov/datastandards/dictionaries/#ctli
The term"clti" is the namespace of the LDD.

@rchenatjpl
Copy link

Thanks. Since 1) I don't have the CTLI Ingest_LDD, and 2) lddtool obviously was able to process it, I'll skip testing this as well.

@jordanpadams
Copy link
Member

@rchenatjpl I think that is fine. we will eventually get our hands on the IngestLDD, but if Lyle and Co tested it, we can consider this good for now.

@jshughes
Copy link
Collaborator Author

I agree, at some time the Ingest_LDDs should under configuration management in GitHub.

@jordanpadams
Copy link
Member

@rchenatjpl just as an FYI, the latest ZIP posted https://pds.nasa.gov/datastandards/dictionaries/ctli/v1/PDS4_CTLI_1D00_1101.zip. not sure you really need to test this, but just a heads up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants