Merge pull request #1239 from nipreps/fix/config-save

FIX: Avert costly ``BIDSLayout.__repr__`` calls when saving config
nipreps · Apr 7, 2024 · 5af71a0 · 5af71a0
2 parents baf56bb + cb3b2b4
commit 5af71a0
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/mriqc/config.py b/mriqc/config.py
@@ -431,7 +431,6 @@ class execution(_Config):
         'bids_database_dir',
         'fs_license_file',
         'fs_subjects_dir',
-        'layout',
         'log_dir',
         'output_dir',
         'templateflow_home',
@@ -511,6 +510,11 @@ def init(cls):
                 indexer=_indexer,
             )
 
+            # Rewrite __repr__ to avoid the layout query and build the summary
+            # For a smallish dataset this takes one minute each time.
+            # See https://github.com/nipreps/mriqc/issues/1239
+            cls._layout.__class__.__repr__ = lambda x: f'BIDS Layout: {cls.bids_dir}'
+
         cls.layout = cls._layout