Skip to content

Commit c56a3b1

Browse files
committed
STY: Reuse comparison, rewrite warning
1 parent 690c620 commit c56a3b1

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

Diff for: nipype/interfaces/base/core.py

+12-12
Original file line numberDiff line numberDiff line change
@@ -277,15 +277,15 @@ def _check_version_requirements(self, trait_object, raise_exception=True):
277277
for name in names:
278278
min_ver = LooseVersion(str(trait_object.traits()[name].min_ver))
279279
try:
280-
min_ver > version
280+
too_old = min_ver > version
281281
except TypeError:
282282
iflogger.warning(
283-
'Nipype is having issues parsing the package version '
284-
f'for Trait {name} ({self.__class__.__name__})'
285-
f'You may want to check whether {version} is larger than {min_ver}'
286-
)
283+
f"Nipype cannot validate the package version {version!r} for "
284+
f"{self.__class__.__name__}. Trait {name} requires version "
285+
f">={min_ver}. Please verify validity."
286+
)
287287
continue
288-
if min_ver > version:
288+
if too_old:
289289
unavailable_traits.append(name)
290290
if not isdefined(getattr(trait_object, name)):
291291
continue
@@ -303,15 +303,15 @@ def _check_version_requirements(self, trait_object, raise_exception=True):
303303
for name in names:
304304
max_ver = LooseVersion(str(trait_object.traits()[name].max_ver))
305305
try:
306-
max_ver > version
306+
too_new = max_ver < version
307307
except TypeError:
308308
iflogger.warning(
309-
'Nipype is having issues parsing the package version '
310-
f'for Trait {name} ({self.__class__.__name__})'
311-
f'You may want to check whether {version} is smaller than {max_ver}'
312-
)
309+
f"Nipype cannot validate the package version {version!r} for "
310+
f"{self.__class__.__name__}. Trait {name} requires version "
311+
f"<={max_ver}. Please verify validity."
312+
)
313313
continue
314-
if max_ver < version:
314+
if too_new:
315315
unavailable_traits.append(name)
316316
if not isdefined(getattr(trait_object, name)):
317317
continue

0 commit comments

Comments
 (0)