bastula / dicompyler

Extensible radiation therapy research platform and viewer for DICOM and DICOM RT
http://www.dicompyler.com
268 stars 100 forks source link

Upgrade to pydicom 0.9.5 #24

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently running dicompyler from source with pydicom 0.9.5 will break. This is 
because pydicom now raises a InvalidDicomError if files are not valid DICOM - 
see 
http://code.google.com/p/pydicom/source/detail?r=fc790f01f517f060baa4f92b80311ab
0e9536c8e
Therefore we can remove our own detection in dicomparser.py and use the 
detection from pydicom.

Additionally, pydicom 0.9.5 fixes the recursion error on private data elements 
which occurs with some vendor data, crashing the DICOM tree view plugin.

Original issue reported on code.google.com by bastula on 12 Nov 2010 at 4:15

GoogleCodeExporter commented 9 years ago
Apparently using pydicom 0.9.5 will also not allow the 2D view to work properly 
due to a different pixel_array object for each image.

See the traceback in http://code.google.com/p/dicompyler/issues/detail?id=22#c22

Currently, until this issue is fixed, the solution is to downgrade to pydicom 
0.9.4.

This will be the highest priority issue until then, as other users have 
reported this issue when running from source using pydicom 0.9.5.

Original comment by bastula on 15 Nov 2010 at 5:28

GoogleCodeExporter commented 9 years ago
This issue was closed by revision ab434c5ad5.

Original comment by bastula on 16 Nov 2010 at 4:11

GoogleCodeExporter commented 9 years ago
Tested with Windows/Mac/Linux using several different vendor's data.

Original comment by bastula on 16 Nov 2010 at 4:12