Daniel-Diaz / matrix

A Haskell native implementation of matrices and their operations.
BSD 3-Clause "New" or "Revised" License
35 stars 31 forks source link

combineRows problems #13

Closed randomlogin closed 10 years ago

randomlogin commented 10 years ago

Something wrong is with combineRows function. image I've tried to investigate the problem, but couldn't find a solution. Somehow first parameter (current column) passed to mapRow is bigger than number of column. With some matrices it works perfectly fine, but sometime it crashes as in the image.

Daniel-Diaz commented 10 years ago

I am investigating this issue. I think the problem comes from the mapRow function, not handling well the internal representation of the matrix. Should be fixed soon.

Daniel-Diaz commented 10 years ago

I have fixed the relevant functions to this issue. Let me know if you have further problems in the future. Thank you.