From 9891217a395966089fdc0cc55b506de13c956edf Mon Sep 17 00:00:00 2001
From: Katherine Heal <katherine.heal@pnnl.gov>
Date: Wed, 23 Oct 2024 15:24:22 -0700
Subject: [PATCH] Add more permissible values for CollectionTypeEnum

---
 src/schema/basic_classes.yaml | 9 ++++++++-
 src/schema/nmdc.yaml          | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/schema/basic_classes.yaml b/src/schema/basic_classes.yaml
index c3f1d15159..8bdca8dd8c 100644
--- a/src/schema/basic_classes.yaml
+++ b/src/schema/basic_classes.yaml
@@ -534,6 +534,7 @@ slots:
     range: CollectionTypeEnum
     description: >-
       The type of collection that describes the data in the collection.
+    required: true
 
   model:
     range: InstrumentModelEnum
@@ -617,7 +618,13 @@ enums:
     #TODO KRH: add more collection types, mappings, and descriptions
       instrument_run:
         description: >-
-          A collection of data generated from a single run of an instrument.
+          A collection of data generation records from a single run of an instrument.
+      technical_replicates:
+        description: >-
+          A collection of data generation records that represent technical replicates of a single sample.
+      fractions:
+        description: >-
+          A collection of data generation records that represent fractions of a single sample.
 
   InstrumentModelEnum:
     permissible_values:
diff --git a/src/schema/nmdc.yaml b/src/schema/nmdc.yaml
index 0bc2691d23..0bd5601be8 100644
--- a/src/schema/nmdc.yaml
+++ b/src/schema/nmdc.yaml
@@ -1138,7 +1138,6 @@ slots:
     description: This property links a database object to the set of configurations within it.
 
   manifest_set:
-    #TODO KRH: Do I need a mix in here?
     mixins: object_set
     range: Manifest
     description: This property links a database object to the set of manifests within it.