invesalius / invesalius3

3D medical imaging reconstruction software
https://invesalius.github.io/
GNU General Public License v2.0
632 stars 287 forks source link

For some DICOM sequences, the spacing isn't retrieved properly #475

Open k5123 opened 2 years ago

k5123 commented 2 years ago

For some DICOM sequences i have gotten here there doesn't seem a proper spacing information available (or it's stored differently), causing the "spacing" variable to be null, resulting in an import failure without clear error message.

The console shows:

RAM_FREE= 73786998784 RAM_TOTAL= 137357676544 Traceback (most recent call last): File "invesalius\gui\import_panel.py", line 165, in OnDblClickTextPanel File "invesalius\gui\import_panel.py", line 188, in LoadDicom File "invesalius\pubsub\pub.py", line 74, in sendMessage File "pubsub\core\publisher.py", line 216, in sendMessage topicObj.publish(msgData) File "pubsub\core\topicobj.py", line 452, in publish self.sendMessage(msgData, topicObj, msgDataSubset) File "pubsub\core\topicobj.py", line 482, in sendMessage listener(data, self, allData) File "pubsub\core\listener.py", line 237, in call cb(kwargs) File "invesalius\control.py", line 909, in OnOpenDicomGroup File "invesalius\control.py", line 998, in OpenDicomGroup UnboundLocalError: local variable 'spacing' referenced before assignment

code in question seems to be here: https://github.com/invesalius/invesalius3/blob/66b749f01cc42c5371333575cd2b8a9ffd009026/invesalius/control.py#L992

tfmoraes commented 2 years ago

@k5123 you are using last InVesalius version (https://github.com/invesalius/invesalius3/releases/tag/v3.1.99998)?

k5123 commented 2 years ago

@k5123 you are using last InVesalius version (https://github.com/invesalius/invesalius3/releases/tag/v3.1.99998)?

Yes i am indeed.

tfmoraes commented 2 years ago

@k5123 please, can you send me some of that dicom files? tfmoraes at cti.gov.br

SKrywinski commented 1 year ago

Any news on this? I am facing the same error...

tfmoraes commented 1 year ago

@k5123 please, can you send me some of that dicom files? tfmoraes at cti.gov.br