-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add external analysis project identifiers slots to WorkflowChain
#47
Add external analysis project identifiers slots to WorkflowChain
#47
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new jgi.analysis prefix needs to be defined in prefixes: in nmdc.yaml
expansion should be https://data.jgi.doe.gov/search?q=
@turbomam I'm surprised to see this passing schema tests using a curie prefix that hasn't been defined.
…n' of https://github.com/microbiomedata/berkeley-schema-fy24 into 11232_add_external_analysis_identifiers_to_WorkflowChain
It looks like this PR does have an expansion for If I comment it out, the build and tests do go to completion, but there is a warning:
|
@aclum I've added the jgi.analysis prefix and URL example |
WorkflowChain
no migration required |
#1123
This PR provides:
-- gold_analysis_project_identifiers
-- jgi_portal_analysis_project_identifiers
jgi_portal_analysis_project_identifiers
slot definition