FIX: Do not error when popping missing scan_length (#679)

* FIX scanlength pop issue * pin sqlalchemhy version * Update setup.cfg Co-authored-by: Chris Markiewicz <[email protected]>
bids-standard · Nov 9, 2020 · 17db824 · 17db824
1 parent 7440d48
commit 17db824
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/bids/analysis/analysis.py b/bids/analysis/analysis.py
@@ -100,7 +100,7 @@ def setup(self, steps=None, drop_na=False, finalize=True, **kwargs):
             step.add_collections(drop_na=drop_na, **selectors)
 
         if finalize:
-            selectors.pop('scan_length')  # see TODO below
+            selectors.pop('scan_length', None)  # see TODO below
             self.finalize(**selectors)
 
     def finalize(self, **kwargs):

diff --git a/setup.cfg b/setup.cfg
@@ -27,7 +27,7 @@ install_requires =
     nibabel >=2.1
     pandas >=0.23
     patsy
-    sqlalchemy
+    sqlalchemy <1.4.0.dev0
     bids-validator
     num2words
     click