Skip to content

Commit 0e83074

Browse files
bpinsardyarikoptic
andcommitted
Apply suggestions from code review
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
1 parent 7de4403 commit 0e83074

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

heudiconv/convert.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,13 +525,12 @@ def update_multiorient_name(
525525
metadata: dict[str, Any],
526526
filename: str,
527527
) -> str:
528-
if "acq-" in filename:
528+
if "_acq-" in filename:
529529
lgr.warning(
530-
"Not embedding multi-orientation information as prefix already uses acq- parameter."
530+
"Not embedding multi-orientation information as `%r` already uses acq- parameter.", filename
531531
)
532532
return filename
533533
iop = metadata.get("ImageOrientationPatientDICOM")
534-
# iop = [round(x) for x in iop]
535534
assert isinstance(iop, list)
536535
cross_prod = [
537536
iop[1] * iop[5] - iop[2] * iop[4],

0 commit comments

Comments
 (0)