microbiomedata / nmdc-schema

National Microbiome Data Collaborative (NMDC) unified data model
https://microbiomedata.github.io/nmdc-schema/
Creative Commons Zero v1.0 Universal
27 stars 8 forks source link

Metabolomics Schema Updates Meta Issue #1905

Closed brynnz22 closed 3 months ago

brynnz22 commented 5 months ago

MaterialProcessing

DataGeneration

WorkflowExecution

Future things to Consider:

turbomam commented 4 months ago

I haven't had a chance to discuss any of these tasks with you in person yet, and I will be at a conference on Wednesday the 17th.

Do you all feel like you understand the modeling principles that will determine how quickly your branches can be merged in (as opposed to requiring a lot of refactoring?)

I think many of the sub-tasks above should be broken down into even smaller sub-tasks. For example, for class-creation sub-task, there may be new slots. I recommend defining those slots and getting them merged as early on as possible.

turbomam commented 4 months ago

Why are you thinking of creating a workflow_version slot? How is the existing version slot inadequate ?

turbomam commented 4 months ago

If you are going to create classes for modeling configuration, please do so in the most generic possible way. We should look for other existing patterns in the schema that associate configuration, settings or objective with a process.

turbomam commented 4 months ago

When pointing the instance of one class to a configuration, I recommend using a generic slot like has_configuration over workflow_configuration_object

turbomam commented 4 months ago

What will the range of SubsamplingProcess' sample_portion slot be?

kheal commented 4 months ago

Why are you thinking of creating a workflow_version slot? How is the existing version slot inadequate ?

Good call @turbomam. @brynnz22 I think the version slot is adequate for our use - it accepts a string so we can say "MetaMS 2.1" and similar there, unless I'm missing something.