Skip to content

New import names in pydicom 1.0 #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
matthew-brett opened this issue Oct 5, 2015 · 0 comments
Closed

New import names in pydicom 1.0 #351

matthew-brett opened this issue Oct 5, 2015 · 0 comments

Comments

@matthew-brett
Copy link
Member

See: https://github.com/darcymason/pydicom/wiki/Porting-to-pydicom-1.0

Need wrapper something like:

try:
    import dicom as pydicom
except ImportError:
    import pydicom

try:
    read_file = pydicom.dicomio.read_file
except AttributeError:
    read_file = pydicom.read_file
@matthew-brett matthew-brett mentioned this issue Oct 5, 2015
Closed
This was referenced Nov 6, 2015
Merged
Merged
matthew-brett added a commit that referenced this issue Nov 7, 2015
@matthew-brett
Merge pull request #379 from effigies/pydicom1.0
668f48b 
MRG: Prepare for pydicom 1.0

pydicom 1.0 will change the import name from dicom to pydicom, and move dicom.read_file to pydicom.dicomio.read_file. Adjust imports accordingly, and rename dicom to pydicom, when imported directly.

Closes #351
grlee77 pushed a commit to grlee77/nibabel that referenced this issue Mar 15, 2016
@matthew-brett
Merge pull request nipy#379 from effigies/pydicom1.0
d6bda1d 
MRG: Prepare for pydicom 1.0

pydicom 1.0 will change the import name from dicom to pydicom, and move dicom.read_file to pydicom.dicomio.read_file. Adjust imports accordingly, and rename dicom to pydicom, when imported directly.

Closes nipy#351
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@matthew-brett