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

Update PLAID to better manage dependencies #18

Open
jordanpadams opened this issue Jan 20, 2021 · 0 comments
Open

Update PLAID to better manage dependencies #18

jordanpadams opened this issue Jan 20, 2021 · 0 comments
Assignees

Comments

@jordanpadams
Copy link
Member

Current implementation

#15 will be implemented using a temporary solution where all discipline LDDs and the PDS common namespace are all in one uber JSON.

Proposed improvement

For a more refined solution, we would like to have each discipline LDD have its own JSON, and PLAID will parse that JSON for a new namespaces key to determine all dependent namespaces for the LDD, and load those other JSONs as appropriate.

Per this improvement to LDDTool, generated LDDs now contain this new key. See attached for the CART LDD: PDS4_CART_1G00_1950.JSON.txt

From this example you can see the new namespaces key:

[
  {
    "dataDictionary": {
      "Title": "PDS4 Data Dictionary" ,
      "Version": "1.16.0.0" ,
      "Date": "Tue Jan 19 19:36:45 PST 2021" ,
      "Description": "This document is a dump of the contents of the PDS4 Data Dictionary" ,
      "namespaces": ["pds:", "geom:", "cart:"] ,
      "classDictionary": [
...

PLAID should use this to then go through the other JSON files and parse each of those namespaces

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants