bug in XYImageItem.set_xy()

[Carlos Pascual]

Hi,
I just found a bug in the set_xy method of XYImageItem class (in image.py of
guiqwt 2.1.3).

I mark it in the following code snippet from image.py:

++++++++++++++++++++++++++++++++++++++++
class XYImageItem(RawImageItem):
(...)
def set_xy(self, x, y):
(...)
if y.shape[0] == ni:
self.y = to_bins(y)
elif x.shape[0] == ni+1: #<<<<<<<< THIS SHOULD BE: y.shape[0] !!!
self.y = y
else:
raise IndexError("y must be a 1D array of length %d or %d" \
% (ni, ni+1))
++++++++++++++++++++++++++++++++++++++++

I hope this helps.

Carlos
--
+----------------------------------------------------+
Carlos Pascual Izarra
Scientific Software Coordinator
Computing Division
Cells / Alba Synchrotron [http:/www.cells.es]
Carretera BP 1413 de Cerdanyola-Sant Cugat, Km. 3.3
E-08290 Cerdanyola del Valles (Barcelona), Spain
E-mail: cpascual-***@public.gmane.org
Phone: [phone]
+----------------------------------------------------+

[Pierre Raybaut]

Hi Carlos,

Thanks for the feedback.

I've just fixed this in the development version.

Cheers,
Pierre

Post by Carlos Pascual
Hi,
I just found a bug in the set_xy method of XYImageItem class (in image.py of
guiqwt 2.1.3).
++++++++++++++++++++++++++++++++++++++++
(...)
(...)
self.y = to_bins(y)
elif x.shape[0] == ni+1: #<<<<<<<< THIS SHOULD BE: y.shape[0] !!!
self.y = y
raise IndexError("y must be a 1D array of length %d or %d" \
% (ni, ni+1))
++++++++++++++++++++++++++++++++++++++++
I hope this helps.
Carlos
--
+----------------------------------------------------+
Carlos Pascual Izarra
Scientific Software Coordinator
Computing Division
Cells / Alba Synchrotron [http:/www.cells.es]
Carretera BP 1413 de Cerdanyola-Sant Cugat, Km. 3.3
E-08290 Cerdanyola del Valles (Barcelona), Spain
Phone: [phone]
+----------------------------------------------------+