FIX: PEP8 - E712 comparison to True/False should be 'if cond is True/False:' or 'if (not) cond:'

Author: miykael

nipype/fixes/numpy/testing/nosetester.py

@@ -245,7 +245,7 @@ def prepare_test_args(self, label='fast', verbose=1, extra_argv=None,
         plugins += [p() for p in nose.plugins.builtin.plugins]
         # add doctesting if required
         doctest_argv = '--with-doctest' in argv
-        if doctests == False and doctest_argv:
+        if doctests is False and doctest_argv:
             doctests = True
         plug = self._get_custom_doctester()
         if plug is None:

nipype/interfaces/cmtk/parcellation.py

@@ -273,7 +273,7 @@ def create_roi(subject_id, subjects_dir, fs_dir, parcellation_name, dilation):
 
     iflogger.info("[ DONE ]")
     # dilate cortical regions
-    if (dilation == True):
+    if dilation is True:
         iflogger.info("Dilating cortical regions...")
         # loop throughout all the voxels belonging to the aseg GM volume
         for j in range(xx.size):
@@ -463,7 +463,7 @@ def crop_and_move_datasets(subject_id, subjects_dir, fs_dir, parcellation_name,
 
     ds.append((op.abspath('ROI_%s.nii.gz' % parcellation_name),
                op.abspath('ROI_HR_th.nii.gz')))
-    if(dilation == True):
+    if dilation is True:
         ds.append((op.abspath('ROIv_%s.nii.gz' % parcellation_name),
                    op.abspath('ROIv_HR_th.nii.gz')))
     orig = op.join(fs_dir, 'mri', 'orig', '001.mgz')
@@ -588,7 +588,7 @@ def _list_outputs(self):
         else:
             outputs['roi_file'] = op.abspath(
                 self._gen_outfilename('nii.gz', 'ROI'))
-        if(self.inputs.dilation == True):
+        if self.inputs.dilation is True:
             outputs['roiv_file'] = op.abspath(self._gen_outfilename(
                 'nii.gz', 'ROIv'))
         outputs['white_matter_mask_file'] = op.abspath('fsmask_1mm.nii.gz')
@@ -597,7 +597,7 @@ def _list_outputs(self):
         outputs['aseg_file'] = op.abspath('aseg.nii.gz')
         outputs['roi_file_in_structural_space'] = op.abspath(
             'ROI_HR_th.nii.gz')
-        if(self.inputs.dilation == True):
+        if self.inputs.dilation is True:
             outputs['dilated_roi_file_in_structural_space'] = op.abspath(
                 'ROIv_HR_th.nii.gz')
         return outputs

nipype/interfaces/freesurfer/preprocess.py

@@ -1003,8 +1003,8 @@ class ApplyVolTransform(FSCommand):
     def _get_outfile(self):
         outfile = self.inputs.transformed_file
         if not isdefined(outfile):
-            if self.inputs.inverse == True:
-                if self.inputs.fs_target == True:
+            if self.inputs.inverse is True:
+                if self.inputs.fs_target is True:
                     src = 'orig.mgz'
                 else:
                     src = self.inputs.target_file

nipype/interfaces/fsl/dti.py

@@ -836,7 +836,7 @@ def _list_outputs(self):
 
         outputs['log'] = os.path.abspath(os.path.join(out_dir, 'probtrackx.log'))
         # utputs['way_total'] = os.path.abspath(os.path.join(out_dir, 'waytotal'))
-        if isdefined(self.inputs.opd == True):
+        if isdefined(self.inputs.opd is True):
             if isinstance(self.inputs.seed, list) and isinstance(self.inputs.seed[0], list):
                 outputs['fdt_paths'] = []
                 for seed in self.inputs.seed:

nipype/interfaces/slicer/generate_classes.py

@@ -227,7 +227,7 @@ def generate_class(module, launcher, strip_module_name_prefix=True, redirect_x=F
                               param.nodeName.replace('-vector', '')]]
                 else:
                     values = [typesDict[param.nodeName.replace('-vector', '')]]
-                if mipav_hacks == True:
+                if mipav_hacks is True:
                     traitsParams["sep"] = ";"
                 else:
                     traitsParams["sep"] = ','

nipype/pipeline/plugins/sgegraph.py

@@ -16,9 +16,9 @@ def node_completed_status(checknode):
     :return: boolean value True indicates that the node does not need to be run.
     """
     """ TODO: place this in the base.py file and refactor """
-    node_state_does_not_require_overwrite = (checknode.overwrite == False or
-                                              (checknode.overwrite == None and
-                                               not checknode._interface.always_run)
+    node_state_does_not_require_overwrite = (checknode.overwrite is False or
+                                              (checknode.overwrite is None and not
+                                               checknode._interface.always_run)
     )
     hash_exists = False
     try:

nipype/pipeline/plugins/slurmgraph.py

@@ -16,9 +16,9 @@ def node_completed_status(checknode):
     :return: boolean value True indicates that the node does not need to be run.
     """
     """ TODO: place this in the base.py file and refactor """
-    node_state_does_not_require_overwrite = (checknode.overwrite == False or
-                                              (checknode.overwrite == None and
-                                               not checknode._interface.always_run)
+    node_state_does_not_require_overwrite = (checknode.overwrite is False or
+                                              (checknode.overwrite is None and not
+                                               checknode._interface.always_run)
     )
     hash_exists = False
     try: