fc13240 / dicompyler

Automatically exported from code.google.com/p/dicompyler
0 stars 0 forks source link

Fix exceptions and try to get rid of dreaded PyDeadObjectError #60

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Load GEANT4/GAMOS three slice thorax CT patient.
2. Noticed FFS images were reversed. 
3. Attempted to open another patient, PyDeadObjectError occurred.

What is the expected output? What do you see instead?
What was once the last slice, 3/3 at -92.5mm, is now the first slice which 
before was 1/3 and located at -82.5mm. This patient is FFS and this DID NOT 
happen with a HFS CT. 

What version of the product are you using? On what operating system?
0.4.1 on Ununtu 10.04 with Python 2.6

Please provide any additional information below.
Attached is the log file, the lockout didn't occur till the very end.

Original issue reported on code.google.com by lstrd...@gmail.com on 1 Jan 2012 at 4:41

Attachments:

GoogleCodeExporter commented 8 years ago
There was a change made in r57d9155cc415 such that images are resorted 
according to the patient position.

Typically in radiation therapy a patient that is scanned in a given orientation 
will be treated in that orientation. This change now adheres to this practice, 
so HFS patients wil be displayed head first and FFS patients will be displayed 
feet first.

In radiology, this is not the case, as all radiologists like to see the patient 
head first regardless of scanning orientation.

Regarding the lockout, by looking at the logs, it seems like the treeview or 
the 2dview might have an issue. I have seen even in previous versions of 
dicompyler that it will get unstable after too many errors (as you can see in 
the log). The only solution is to knock out each of these errors. In the 
meantime, the best option is to close dicompyler after you start getting too 
many errors.

Original comment by bastula on 1 Jan 2012 at 5:46

GoogleCodeExporter commented 8 years ago
Fixed in rb4409744e84c.

Original comment by bastula on 1 May 2012 at 2:23