Opened 12 years ago
Closed 12 years ago
#838 closed defect (fixed)
matrix-utils: cons called with one argument
Reported by: | sjamaan | Owned by: | Ivan Raikov |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | extensions | Version: | 4.7.x |
Keywords: | Cc: | ||
Estimated difficulty: |
Description
The matrix-utils egg has two places where it's calling the Scheme cons
procedure with only one argument. From the looks of it, there's a y
that accidentally got erased (there are two spaces between cons
and b
):
Index: matrix-utils.scm =================================================================== --- matrix-utils.scm (revision 26685) +++ matrix-utils.scm (working copy) @@ -293,7 +293,7 @@ (f (vector-ref A (fx+ (cdr x+b) y)) ax) ax))))))) ((= order ColMajor) - (fold-ec x0 (:parallel (:range b (fx* N ix) (fx* N M) M) (:range y iy ey)) (cons b) + (fold-ec x0 (:parallel (:range b (fx* N ix) (fx* N M) M) (:range y iy ey)) (cons y b) (lambda (y+b ax) (fold-ec ax (:range x ix ex) x (lambda (x ax) @@ -344,7 +344,7 @@ (f i j (vector-ref A (fx+ (cdr x+b) y)) ax) ax)))))) ((= order ColMajor) - (fold-ec x0 (:parallel (:range b (fx* N ix) (fx* N M) M) (:range y iy ey)) (cons b) + (fold-ec x0 (:parallel (:range b (fx* N ix) (fx* N M) M) (:range y iy ey)) (cons y b) (lambda (y+b ax) (fold-ec ax (:range x ix ex) x (lambda (x ax)
The above patch has NOT been tested. I don't have the BLAS library installed, so matrix-utils cannot be installed either as it depends on the blas egg.
This was found by Mario's scrutiny run, see http://parenteses.org/mario/misc/specialize-report/install/matrix-utils.html
Change History (2)
comment:1 Changed 12 years ago by
Milestone: | 4.8.0 |
---|
comment:2 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Fixed in release 1.13 of matrix-utils, thanks for reporting.