pombreda / dicompyler

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

Support matplotlib 1.0.0 and above #39

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In the 2dview, the isodose contour generation fails when using matplotlib 1.0.0 
or above due to a change in the function arguments.

Traceback from Steve Terry:

Traceback (most recent call last):
 File "C:\Documents and Settings\flounder\Desktop\dicompyler
\baseplugins\2dview
.py", line 426, in OnPaint
   self.DrawIsodose(isodose, gc, isodosegen)
 File "C:\Documents and Settings\flounder\Desktop\dicompyler
\baseplugins\2dview
.py", line 279, in DrawIsodose
   contours = isodosegen.trace(level, points = 0)
TypeError: 'points' is an invalid keyword argument for this function

The change in matplotlib source is documented here: 
http://matplotlib.svn.sourceforge.net/viewvc/matplotlib/trunk/matplotlib/src/cnt
r.c?r1=7341&r2=7342

The workaround is to make a conditional to support matplotlib versions 0.99 and 
below and versions 1.0.0 and above.

Original issue reported on code.google.com by bastula on 6 Apr 2011 at 8:06

GoogleCodeExporter commented 9 years ago
The issue was fixed in r d837dbb6312b.

However, the axes are still not scaled properly when using matplotlib 1.0.0 and 
above. See the attached screenshots for an example in the DVH View. Although 
the graphs are still correct, the axes need to be scaled further.

Additionally, when using matplotlib 1.0.0, when right clicking on the DVH View, 
you cannot zoom back out of the graph.

The above issues may have to do with changes that break wxmpl.

Original comment by bastula on 6 Apr 2011 at 10:02

Attachments:

GoogleCodeExporter commented 9 years ago
This issue was updated by revision a52e7bcf982e.

The right click functionality still does not work on matplotlib 1.0.0 and above.

Original comment by bastula on 18 Apr 2011 at 3:28

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

Original comment by bastula on 18 Apr 2011 at 3:52