fix: Load subjects

effigies · effigies · commit f9e214dd524c · 2025-02-13T15:54:57.000-05:00
Type checker doesn't like it, but this fixes it for now.

The real fix I think will be to only call the function once, so we can
get rid of the `null` check.
diff --git a/src/schema/context.ts b/src/schema/context.ts
@@ -63,11 +63,8 @@ export class BIDSContextDataset implements Dataset {
           ?.filter((ext) => ext.endsWith('/'))
         : [],
     )
-    this.subjects = args.subjects || {
-      sub_dirs: this.tree.directories.map((dir) => dir.name).filter((dir) =>
-        dir.startsWith('sub-')
-      ),
-    }
+    // @ts-ignore
+    this.subjects = args.subjects || null
   }
 
   get dataset_description(): Record<string, unknown> {

Original file line number	Diff line number	Diff line change
`@@ -63,11 +63,8 @@ export class BIDSContextDataset implements Dataset {`
`63`	`63`	`?.filter((ext) => ext.endsWith('/'))`
`64`	`64`	`: [],`
`65`	`65`	`)`
`66`		`- this.subjects = args.subjects \|\| {`
`67`		`- sub_dirs: this.tree.directories.map((dir) => dir.name).filter((dir) =>`
`68`		`- dir.startsWith('sub-')`
`69`		`- ),`
`70`		`- }`
	`66`	`+ // @ts-ignore`
	`67`	`+ this.subjects = args.subjects \|\| null`
`71`	`68`	`}`
`72`	`69`
`73`	`70`	`get dataset_description(): Record<string, unknown> {`