Skip to content

Commit

Permalink
Merge pull request #782 from yarikoptic/bf-pydicom
Browse files Browse the repository at this point in the history
Fixup testing: kludge for pydicom 3.0.0 in dcmstack, ignore some warnings from nipype for python 3.12
  • Loading branch information
yarikoptic committed Sep 13, 2024
2 parents ca24297 + bf793c9 commit 1b0bb5a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
5 changes: 5 additions & 0 deletions heudiconv/dicoms.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,11 @@
# suppress warning
import nibabel.nicom.dicomwrappers as dw

# TODO: remove the kludge whenever
# https://github.com/moloney/dcmstack/pull/90 is merged and released
if not hasattr(dcm, "read_file"):
dcm.read_file = dcm.dcmread

lgr = logging.getLogger(__name__)
total_files = 0
# Might be monkey patched by user heuristic to tune desired compression level.
Expand Down
2 changes: 2 additions & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ filterwarnings =
ignore:.*pkg_resources:DeprecationWarning
# <https://github.com/nipy/nipype/issues/3563>
ignore:.*Use setlocale.* instead:DeprecationWarning:nipype
# <https://github.com/nipy/nipype/pull/3670>
ignore:.*datetime.datetime.utcnow\(\) is deprecated.*:DeprecationWarning:nipype

[coverage:run]
include = heudiconv/*
Expand Down

0 comments on commit 1b0bb5a

Please sign in to comment.