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

[namespace-registry] add new namespace "ml" #353

Closed
2 tasks done
jordanpadams opened this issue May 14, 2021 · 4 comments
Closed
2 tasks done

[namespace-registry] add new namespace "ml" #353

jordanpadams opened this issue May 14, 2021 · 4 comments
Assignees

Comments

@jordanpadams
Copy link
Member

jordanpadams commented May 14, 2021

  • Namespace ID Requested: ml (lower case ML)
  • Discipline or Mission LDD? discipline
  • Steward Organization Name: PDS IMG
  • LDD Steward Name: Michael Mcauley
  • LDD Steward Email: Michael.McAuley@jpl.nasa.gov
  • LDD Description: Machine Learning Classifier Discipline Local Data Dictionary
  • Rationale for creation of new LDD: Specifying metadata generated by machine learning algorithms associated with PDS observational products

Engineering Details

@jordanpadams
Copy link
Member Author

@myche @wkiri FYI. this is per this warning you will get when you generate your LDD:

>>> WARNING Init:  - Config.Properties Namespace Id Not Found:ml

@myche
Copy link
Contributor

myche commented May 14, 2021

Hmm .. OK. Does this mean that the ml namespace needs to be added to some master LDD config file somewhere?

... wait, I may have just answered my own question: I see a task item above refering to exactly this.

jshughes added a commit that referenced this issue May 17, 2021
…, ml

Add the namespaces clementine, ctli, mars2020, and ml  to the namespace registry and the config.properties file.

Resolves #332, #344, #352, #353
jshughes added a commit that referenced this issue May 17, 2021
…, ml (#354)

Add the namespaces clementine, ctli, mars2020, and ml  to the namespace registry and the config.properties file.

Resolves #332, #344, #352, #353
@jordanpadams
Copy link
Member Author

updated per #354 . still in dev phase so no need to add to online config yet

@jordanpadams
Copy link
Member Author

@myche correct. in order to avoid the warning about the missing namespace, we need to add this to an LDDTool internal config

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

No branches or pull requests

4 participants