Add cloud laboratory protocol value sets#63
Open
cmungall wants to merge 3 commits into
Open
Conversation
Add lab_automation/cloud_lab.yaml with two controlled vocabularies modeled on the Emerald Cloud Lab / Symbolic Lab Language operational taxonomy: - LabUnitOperationEnum: composable sample-preparation unit operations (transfer, aliquot, mix, incubate, filter, centrifuge, pellet, resuspend, lyophilize, etc.), complementing lab_automation/operations. - CloudLabExperimentEnum: higher-level experiment/assay functions grouped by category (synthesis, separations, spectroscopy, mass spectrometry, bioassays, crystallography, property measurement, cellular), cross-referenced to bio/assays. Wire the module into the top-level valuesets.yaml import list. Ontology meaning: mappings to OBI/CHMO/MMO are being added in a follow-up. https://claude.ai/code/session_01G74MMk5js9HDsGL3VbfAxQ
Map permissible values to ontology terms, every CURIE verified via OAK/OLS against CHMO and OBI: - LabUnitOperationEnum: mix, centrifuge, filter, degas, evaporate, lyophilize, grind. - CloudLabExperimentEnum: chromatography/separations, electrophoresis, spectroscopy, mass spectrometry, and core bioassays (ELISA, qPCR, Western blot, DNA sequencing, PCR, BLI). Add aliases: entries equal to the exact ontology label where it differs in form from the title (verb vs noun, acronyms, en-dash variants) to keep mappings label-consistent. Workflow/labeling primitives and values without a precise, label-consistent ontology class are intentionally left unmapped. https://claude.ai/code/session_01G74MMk5js9HDsGL3VbfAxQ
Contributor
|
Claude finished @cmungall's task —— View job Code Review Complete ✅Reviewing this PR that adds cloud laboratory protocol value sets. Here's my review checklist:
Review SummaryThis is a high-quality contribution that adds comprehensive controlled vocabularies for cloud laboratory automation. The implementation follows project conventions and best practices. ✅ StrengthsSchema Design & Structure:
Ontology Integration:
Documentation & Metadata:
🔍 Minor ObservationsConsistency:
Coverage:
🚀 No Issues Found
📋 Recommendations
This PR significantly enhances the value set collection with well-structured, properly annotated laboratory automation vocabularies. Recommend approval. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds comprehensive controlled vocabularies for cloud-laboratory research protocols, modeled on the operational taxonomy of remote/cloud labs such as Emerald Cloud Lab (ECL) and its Symbolic Lab Language (SLL).
Key Changes
New schema module:
src/valuesets/schema/lab_automation/cloud_lab.yamlLabUnitOperationEnum: 40+ composable sample-manipulation primitives (Transfer, Aliquot, Mix, Incubate, Filter, Centrifuge, etc.)CloudLabExperimentEnum: 100+ higher-level experiment/assay functions organized by category (synthesis, separations/chromatography, spectroscopy, mass spectrometry, bioassays, crystallography, property measurement, cellular)meaning:fields where equivalent classes existUpdated main schema:
src/valuesets/schema/valuesets.yamllab_automation/cloud_labmoduleUpdated ontology term caches:
cache/chmo/terms.csvcache/obi/terms.csvImplementation Details
bio/assays/(OBIAssayEnum, BAOBioassayEnum)https://claude.ai/code/session_01G74MMk5js9HDsGL3VbfAxQ