updates to MagsAnalysis slots

[aclum]

Add descriptions, mappings, minimum values on slots for class MagsAnalysis. Add an enumeration as the range to bin_quality. There is not currently any data in the schema that uses these slots. There may before the November release, all that should conform to these new rules but. I will add to November release notes that workflow_execution_set should be checked with linkml-validate as part of the release process. Tagging @eecavanna @pkalita-lbl for awareness.

PR Information

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Optimization
• Documentation
• Schema change: Structure and content
  • created, updated, or deleted a class, slot, or enum
  • changed whether a slot is multivalued
  • changed the way a slot is assigned to a class
  • changed the permissible_values of an enum
  • etc.
• Schema change: Cleanup and preparation
  • updated the description of a class, slot, or enum
  • updated the mappings of a class, slot, or enum to an ontology
  • added an enum for future use (it is not in the range of any slot)
  • etc.

Description

PRs should be small and concise.

Aim to create small, focused pull requests that fulfill a single purpose. Smaller pull requests are easier and faster to review and merge, leave less room to introduce bugs, and provide a clearer history of changes.

• Replace this text with a description of what this PR branch contains. Please keep in mind that all reviewers will be reading this description. Example: "In this branch, I..."

Related Issues

All PRs should relate to or fix an issue(s). Please identify the issue(s) below.

• Related Issue(s): #
• Fixes: Other updates to MagBin modeling #2160

Did you add/update any tests?

• Yes
• No (Add a justification below)
• I need help with writing tests

Could this schema change make it so any valid data becomes invalid?

This is a question about what the schema allows. It is not a question about what happens to exists in the NMDC database right now.

Example: If, in this PR branch, you renamed a slot from foo to foo_bar, the answer to this question would be "yes," even if nothing in the NMDC database currently uses the foo slot.

More examples: slot or class name changes, changes to a slot's multivalued state, changes to a slot's range (e.g. string to integer), changes to slot assignments to classes, changes to an enum's permissible_values

• Yes (A migrator is required)
• No
• I need help determining this

If you answered "Yes", does this PR branch include that migrator?

• Yes
• No, this PR is incomplete and I need help writing the migrator

Does this PR have any downstream implications?

Examples: any change here that requires a change to workflows, workflow automation, the Mongo-to-Postgres ingest process, Jupyter notebooks, the Runtime, etc.

• Yes (Explain below)
• No

[aclum]

@turbomam can you point me to an for setting a range to only positive integers?

[github-actions]

PR Preview Action v1.4.8
🚀 Deployed preview to https://microbiomedata.github.io/nmdc-schema/pr-preview/pr-2225/
on branch gh-pages at 2024-10-21 18:09 UTC