From d57adba39a69ded3c49edf4657e396261970ad58 Mon Sep 17 00:00:00 2001
From: Evan Edmond <evan.edmond@ndcn.ox.ac.uk>
Date: Wed, 25 Nov 2020 13:26:49 +0000
Subject: [PATCH] ENH: <scanner> defaults to 'SIEMENS'

---
 fsl/wrappers/fugue.py                | 8 +++++---
 tests/test_wrappers/test_wrappers.py | 3 +--
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/fsl/wrappers/fugue.py b/fsl/wrappers/fugue.py
index b98841fd..e7ff7bd8 100644
--- a/fsl/wrappers/fugue.py
+++ b/fsl/wrappers/fugue.py
@@ -86,9 +86,11 @@ def prelude(**kwargs):
 
 @wutils.fileOrImage('phase_image', 'magnitude_image', 'out_image')
 @wutils.fslwrapper
-def fsl_prepare_fieldmap(scanner, phase_image, magnitude_image, out_image, deltaTE, **kwargs):
-    """Wrapper for the ``fsl_prepare_fieldmap`` command."""
-        
+def fsl_prepare_fieldmap(phase_image, magnitude_image, out_image, deltaTE, scanner=None, **kwargs):
+    """Wrapper for the ``fsl_prepare_fieldmap`` command. <scanner> defaults to 'SIEMENS'"""
+    if scanner is None:
+        scanner = 'SIEMENS'
+    
     valmap = {
         'nocheck' : wutils.SHOW_IF_TRUE,
     }
diff --git a/tests/test_wrappers/test_wrappers.py b/tests/test_wrappers/test_wrappers.py
index a85c4b16..dbffff1d 100644
--- a/tests/test_wrappers/test_wrappers.py
+++ b/tests/test_wrappers/test_wrappers.py
@@ -389,8 +389,7 @@ def test_tbss():
 def test_fsl_prepare_fieldmap():
     with asrt.disabled(), run.dryrun(), mockFSLDIR(bin=('fsl_prepare_fieldmap',)) as fsldir:
         fpf = op.join(fsldir, 'bin', 'fsl_prepare_fieldmap')
-        result   = fw.fsl_prepare_fieldmap(scanner='SIEMENS', 
-                                                 phase_image='ph', 
+        result   = fw.fsl_prepare_fieldmap(phase_image='ph', 
                                                  magnitude_image='mag', 
                                                  out_image='out', 
                                                  deltaTE=2.46, 
-- 
GitLab