Merge branch 'main' into bruker-single-converter

schemas/base-metadata.schema.ts

@@ -1,6 +1,14 @@
 import baseMetadataSchema from './json/base_metadata_schema.json' assert { type: "json" }
 
-baseMetadataSchema.properties.Subject.properties.weight.unit = 'kg' // Add unit to weight
+export const preprocessMetadataSchema = (schema: any = baseMetadataSchema) => {
 
+    // Add unit to weight
+    schema.properties.Subject.properties.weight.unit = 'kg'
 
-export default baseMetadataSchema
+    // Override description of keywords
+    schema.properties.NWBFile.properties.keywords.description = 'Terms to describe your dataset (e.g. Neural circuits, V1, etc.)' // Add description to keywords
+    return schema
+
+}
+
+export default preprocessMetadataSchema()

schemas/json/base_metadata_schema.json

@@ -102,7 +102,7 @@
         },
         "stimulus_notes": {
           "type": "string",
-          "description": "Notes about data collection and analysis."
+          "description": "Notes about stimuli, such as how and where presented."
         }
       }
     },

schemas/source-data.schema.ts

@@ -1,10 +1,18 @@
 // import { merge } from "../src/renderer/src/stories/pages/utils"
 
-export default function getSourceDataSchema (schema) {
-    // const copy = merge(schema, {})
+export default function preprocessSourceDataSchema (schema) {
+
+    // Abstract across different interfaces
     Object.values(schema.properties ?? {}).forEach((schema: any) => {
-        if (schema.properties.gain) schema.properties.gain.step = null // Do not show steps
+
+            // Do not show steps
+            if (schema.properties.gain) schema.properties.gain.step = null
+
+            // Add description to exclude_cluster_groups
+            if (schema.properties.exclude_cluster_groups) schema.properties.exclude_cluster_groups.description = 'Cluster groups to exclude (e.g. "noise" or ["noise", "mua"]).'
+
     })
 
+
     return schema
 }

src/renderer/src/stories/pages/guided-mode/data/GuidedMetadata.js

@@ -11,6 +11,7 @@ import { SimpleTable } from "../../../SimpleTable.js";
 import { onThrow } from "../../../../errors";
 import { merge } from "../../utils.js";
 import { NWBFilePreview } from "../../../preview/NWBFilePreview.js";
+import { preprocessMetadataSchema } from "../../../../../../../schemas/base-metadata.schema";
 
 const getInfoFromId = (key) => {
     let [subject, session] = key.split("/");
@@ -106,7 +107,7 @@ export class GuidedMetadataPage extends ManagedPage {
         const form = new JSONSchemaForm({
             identifier: instanceId,
             mode: "accordion",
-            schema,
+            schema: preprocessMetadataSchema(schema),
             results,
             globals: aggregateGlobalMetadata,
 

src/renderer/src/stories/pages/guided-mode/data/GuidedSourceData.js

@@ -6,7 +6,7 @@ import { ManagedPage } from "./ManagedPage.js";
 import { baseUrl } from "../../../../globals.js";
 import { onThrow } from "../../../../errors";
 import { merge } from "../../utils.js";
-import getSourceDataSchema from "../../../../../../../schemas/source-data.schema";
+import preprocessSourceDataSchema from "../../../../../../../schemas/source-data.schema";
 
 export class GuidedSourceDataPage extends ManagedPage {
     constructor(...args) {
@@ -109,12 +109,10 @@ export class GuidedSourceDataPage extends ManagedPage {
         const schema = this.info.globalState.schema.source_data;
         delete schema.description;
 
-        const schemaResolved = getSourceDataSchema(schema);
-
         const form = new JSONSchemaForm({
             identifier: instanceId,
             mode: "accordion",
-            schema: schemaResolved,
+            schema: preprocessSourceDataSchema(schema),
             results: info.source_data,
             emptyMessage: "No source data required for this session.",
             ignore: [