-
Notifications
You must be signed in to change notification settings - Fork 3
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
Documentation issue - MIxS update #5
Comments
Currently, from what I can tell, an instance of MIxS v5 was pulled and added to the schema. This should be removed. MIxS v6 read the docs should be pulled from MIxS instead of copying an instance. @cmungall , @wdduncan ... can you chime in if you agree with this / provide some context on what we need to do to accomplish this? |
@mslarae13 Has MIXS 6 been released yet? I didn't think it had. Once it is released, our plan is to update the schema with the new version. |
Moving documentation issue to April per @kwdavenport |
@turbomam I think this should be assigned to you? I have NO idea where the MIxS v6 documentation is, or how we will pull it in. If you disagree, we should disucss soon to ensure the documentation gets completed for the April release. |
@turbomam I will re-assign to you. |
I'm going to start collecting some notes here. I don't mean to say that this is all the documentation I'll be providing!. The mechanism for integrating MIxS terms, specifically those from 12 different packages, is briefly described in nmdc-schema PR #292 Especially see the quick and dirty code that does the integration, The process is started by examining all slots included by the previous, essentially static MIxS5 import, I can include/check all slots for a MIMS sample, like the environmental section, etc. (but not the grouping slots themselves!)
like
|
Rough stream of consciousness draft: Integration of MIxS into the NMDC schema: tech notes |
@kwdavenport can we consider this issue closed? |
I don't think so. I'll be talking with Mark and Montana tomorrow. I am thinking it should go into the May sprint, but I can let you know after our meeting (at 8am PT).
…________________________________
From: ssarrafan ***@***.***>
Sent: Thursday, April 28, 2022 3:54 PM
To: microbiomedata/NMDC_documentation
Cc: Davenport, Karen Walston; Mention
Subject: [EXTERNAL] Re: [microbiomedata/NMDC_documentation] Documentation issue - MIxS update (Issue #5)
@kwdavenport<https://urldefense.com/v3/__https://github.com/kwdavenport__;!!Bt8fGhp8LhKGRg!TsI9X1gsBEovJqXiPDvYgiuQgZuS3aQp9cUyeIzLbtni78F9UOhLlqF7ypC2qREu1MM$> can we consider this issue closed?
—
Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https://github.com/microbiomedata/NMDC_documentation/issues/5*issuecomment-1112689037__;Iw!!Bt8fGhp8LhKGRg!TsI9X1gsBEovJqXiPDvYgiuQgZuS3aQp9cUyeIzLbtni78F9UOhLlqF7ypC2n3fEYTc$>, or unsubscribe<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ACOC2VD5Q6C7MEGRTC5M2SDVHMCKZANCNFSM5NH76OLA__;!!Bt8fGhp8LhKGRg!TsI9X1gsBEovJqXiPDvYgiuQgZuS3aQp9cUyeIzLbtni78F9UOhLlqF7ypC2NpOHPkk$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I haven't heard back so I will move this to the May sprint @kwdavenport @turbomam |
How can we get the best, most automated integration between @sujaypatil96's https://microbiomedata.github.io/mixs and https://nmdc-docs-sphinx.readthedocs.io/en/latest/reference/metadata/combined_schema_docs.html#mixs-soil-package (for example), which is managed by @kwdavenport and Yan? |
@ssarrafan : There are Makefile steps for ongoing integration of MIxS terms into the NMDC schema, but it should be automated I haven't really done anything to contribute to the NMDC documentation pages yet. The MIxS content seems like a mixture of narrative content with some cut-and-past that may not be completely accurate any more. |
I'll move this to July then but let me know if you are not planning to work on it and I can move it to the backlog. I'm not sure how high the priority for this is? @cmungall @emileyfadrosh |
This issue has been in multiple sprints. I'm going to move it to the backlog. Let me know if it should be moved to the active sprint. @turbomam @mslarae13 |
@shreddd if we discuss documentation at SW Best Practices this is a good issue to refer to |
@turbomam @mslarae13 is this still needed? |
This looks quite old, so I am going to close out. |
old or young... I am pretty sure NMDC is using the last "official" release of MIxS, and not v6. |
Update MIxS to use v6
The text was updated successfully, but these errors were encountered: