nci-cidc-schemas 0.27.6__tar.gz → 0.27.8__tar.gz

{nci_cidc_schemas-0.27.6/nci_cidc_schemas.egg-info → nci_cidc_schemas-0.27.8}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: nci_cidc_schemas
-Version: 0.27.6
+Version: 0.27.8
 Summary: The CIDC data model and tools for working with it.
 Home-page: https://github.com/NCI-CIDC/cidc-schemas
 Author: NCI
@@ -37,6 +37,7 @@ Dynamic: description-content-type
 Dynamic: home-page
 Dynamic: keywords
 Dynamic: license
+Dynamic: license-file
 Dynamic: requires-dist
 Dynamic: requires-python
 Dynamic: summary

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/cidc_schemas/__init__.py

@@ -2,4 +2,4 @@
 
 __author__ = """NCI"""
 __email__ = "nci-cidc-tools-admin@mail.nih.gov"
-__version__ = "0.27.06"
+__version__ = "0.27.08"

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/cidc_schemas/schemas/sample.json

@@ -493,7 +493,7 @@
         "quality_of_sample": {
             "description": "Final status of sample after QC and pathology review.",
             "type": "string",
-            "enum": ["Pass", "Pass at Risk", "Fail"]
+            "enum": ["QC Pass", "Pass at Risk", "QC Failed", "QC Lost", "QC Not Shipped"]
         },
         "sample_replacement": {
             "description": "Indication if sample replacement is/was requested.",

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/cidc_schemas/schemas/templates/analyses/wes_analysis_template.json

@@ -293,63 +293,63 @@
                             "type_ref": "sample.json#properties/cimac_id",
                             "process_as": [
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/align/{id}/{id}.sorted.dedup.bam'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/align/{run_id}/{normal_id}.sorted.dedup.bam'",
                                     "merge_pointer": "/normal/alignment/align_sorted_dedup",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/sorted.dedup.bam",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/align/{id}/{id}.sorted.dedup.bam.bai'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/align/{run_id}/{normal_id}.sorted.dedup.bam.bai'",
                                     "merge_pointer": "/normal/alignment/align_sorted_dedup_index",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/sorted.dedup.bam.bai",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/align/{id}/{id}_recalibrated.bam'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/align/{run_id}/{normal_id}_recalibrated.bam'",
                                     "merge_pointer": "/normal/alignment/align_recalibrated",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/recalibrated.bam",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/align/{id}/{id}_recalibrated.bam.bai'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/align/{run_id}/{normal_id}_recalibrated.bam.bai'",
                                     "merge_pointer": "/normal/alignment/align_recalibrated_index",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/recalibrated.bam.bai",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/germline/{id}/{id}_haplotyper.output.vcf'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/germline/{run_id}/{normal_id}_haplotyper.output.vcf'",
                                     "merge_pointer": "/normal/germline/haplotyper_output",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/haplotyper_output.vcf",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/germline/{id}/{id}_haplotyper.targets.vcf.gz'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/germline/{run_id}/{normal_id}_haplotyper.targets.vcf.gz'",
                                     "merge_pointer": "/normal/germline/haplotyper_targets",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/haplotyper_targets.vcf.gz",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/hlahd/{id}/result/{id}_final.result.txt'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/hlahd/{run_id}/result/{normal_id}_final.result.txt'",
                                     "merge_pointer": "/normal/hla/hla_final_result",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/hla_final_result.txt",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/optitype/{id}/{id}_result.tsv'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/optitype/{run_id}/{normal_id}_result.tsv'",
                                     "merge_pointer": "/normal/hla/optitype_result",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/optitype_result.tsv",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",
                                     "is_artifact": 1
                                 },
                                 {
-                                    "parse_through": "lambda id: f'{folder or \"\"}analysis/xhla/{id}/report-{id}-hla.json'",
+                                    "parse_through": "lambda normal_id, run_id: f'{folder or \"\"}analysis/xhla/{run_id}/report-{normal_id}-hla.json'",
                                     "merge_pointer": "/normal/hla/xhla_report_hla",
                                     "gcs_uri_format": "{protocol identifier}/wes/{run id}/analysis/normal/{normal cimac id}/xhla_report_hla.json",
                                     "type_ref": "assays/components/local_file.json#properties/file_path",

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/cidc_schemas/template.py

@@ -750,7 +750,7 @@ class _FieldDef(NamedTuple):
 
         if self.parse_through:
             try:
-                raw_val = eval(self.parse_through, format_context, {})(raw_val)
+                raw_val = self._eval_lambda_expression(raw_val, format_context)
 
             # catching everything, because of eval
             except Exception as e:
@@ -786,6 +786,41 @@ class _FieldDef(NamedTuple):
         else:
             return [AtomicChange(self.merge_pointer, val)], files
 
+    def _eval_lambda_expression(self, raw_val: str, format_context: dict):
+        """
+        Evaluates a lambda expression stored in `self.parse_through`, optionally
+        passing an additional parameter if the lambda function has two parameters.
+
+        Args:
+            raw_val (str): Initial value to process.
+            format_context (dict): Dictionary providing format context.
+
+        Returns:
+            Processed value after applying the lambda function.
+        """
+        parse_lambda = self.parse_through
+        # Check for multiple lambda parameters
+        if (
+            parse_lambda.startswith("lambda ")
+            and ":" in parse_lambda
+            and "," in parse_lambda
+        ):
+            # Extract the parameter section (between "lambda " and ":")
+            colon_index = parse_lambda.find(":")
+            param_section = parse_lambda[7:colon_index].strip()
+            params = [param.strip() for param in param_section.split(",")]
+
+            # Handles up to two parameters at this time
+            if len(params) == 2:
+                # Convert underscores to spaces to match column header formatting
+                param_name = params[1].replace("_", " ")
+                extra_val = format_context.get(param_name)
+                # Evaluate the lambda function with two parameters
+                return eval(parse_lambda, format_context, {})(raw_val, extra_val)
+
+        # Evaluate the lambda function with one parameter
+        return eval(parse_lambda, format_context, {})(raw_val)
+
     ## TODO easy - split val coerce / files calc to handle exceptions separately
     ## TODO hard - files (artifact and multi) should be just coerce?
     def _calc_val_and_files(self, raw_val, format_context: dict, encrypt_fn: Callable):

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8/nci_cidc_schemas.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: nci_cidc_schemas
-Version: 0.27.6
+Version: 0.27.8
 Summary: The CIDC data model and tools for working with it.
 Home-page: https://github.com/NCI-CIDC/cidc-schemas
 Author: NCI
@@ -37,6 +37,7 @@ Dynamic: description-content-type
 Dynamic: home-page
 Dynamic: keywords
 Dynamic: license
+Dynamic: license-file
 Dynamic: requires-dist
 Dynamic: requires-python
 Dynamic: summary

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/tests/data/clinicaltrial_examples/CT_1PA_multiWES.json

@@ -116,7 +116,7 @@
                     "material_remaining": 0.0,
                     "material_remaining_units": "Milliliters",
                     "material_storage_condition": "Ambient",
-                    "quality_of_sample": "Pass",
+                    "quality_of_sample": "QC Pass",
                     "sample_replacement": "Replacement Not Requested",
                     "residual_sample_use": "Not Reported"
                 },
@@ -149,7 +149,7 @@
                     "material_remaining": 0.0,
                     "material_remaining_units": "Milliliters",
                     "material_storage_condition": "Ambient",
-                    "quality_of_sample": "Pass",
+                    "quality_of_sample": "QC Pass",
                     "sample_replacement": "Replacement Not Requested",
                     "residual_sample_use": "Not Reported"
                 },
@@ -182,7 +182,7 @@
                     "material_remaining": 0.0,
                     "material_remaining_units": "Milliliters",
                     "material_storage_condition": "Ambient",
-                    "quality_of_sample": "Pass",
+                    "quality_of_sample": "QC Pass",
                     "sample_replacement": "Replacement Not Requested",
                     "residual_sample_use": "Not Reported"
                 }

{nci_cidc_schemas-0.27.6 → nci_cidc_schemas-0.27.8}/tests/data/clinicaltrial_examples/CT_cytof_with_analysis.json

@@ -185,7 +185,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "a1",
           "parent_sample_id": "qwe",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 0,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Microliters",
@@ -210,7 +210,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "a1",
           "parent_sample_id": "qwe",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 0,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Microliters",
@@ -242,7 +242,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "A1",
           "parent_sample_id": "TRIALGROUP 1",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 1,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Slides",
@@ -268,7 +268,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "A2",
           "parent_sample_id": "TRIALGROUP 2",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 1,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Slides",
@@ -294,7 +294,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "A3",
           "parent_sample_id": "TRIALGROUP 3",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 0,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Slides",
@@ -327,7 +327,7 @@
           "type_of_sample": "Fresh Tissue",
           "sample_location": "A4",
           "parent_sample_id": "TRIALGROUP 4",
-          "quality_of_sample": "Pass",
+          "quality_of_sample": "QC Pass",
           "material_remaining": 0,
           "sample_replacement": "Replacement Not Requested",
           "material_used_units": "Slides",