-
Notifications
You must be signed in to change notification settings - Fork 1
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 JGI metagenomics long read class to submission schema #185
Add JGI metagenomics long read class to submission schema #185
Conversation
…submission-schema
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.
do you know if we require the descriptions in the schemasheets classes.tsv file to be in a particular format? It looks like I set a pretty un-helpful precedent. Maybe we should start with descriptions that are more meaningful to humans in this PR. I could update the other ones later this week.
I don't think anything downstream relies on any particular description format, so yeah they should be more descriptive. See also: #186. I'll add something better for the classes added here. |
@turbomam Descriptions updated. Any other feedback? |
Summary
JgiMgLrInterface
class. It is largely similar to the existingJgiMgInterface
class, but it represents metadata for samples to go for long-read sequencing. A few slots have different required/recommended values inJgiMgLrInterface
as compared toJgiMgInterface
.AnalysisTypeEnum
is removed. It's not needed here since it is imported fromnmdc-schema
.nmdc-schema
dependency is bumped up to v10.1.0.