Arity warning for doseq-indexed macro with Clojurescript

[mars0i]

It looks like there's an issue with the doseq-indexed macro in utils.cljc when running Clojurescript. My guess is that this is because the macro is defined in the same file in which it's used, and there are two implicit arguments that the macro expects. I believe that Clojurescript needs macro and function defs to be in different files/namespaces.

Here's an illustration of the problem running lein figwheel from a fresh aljabr clone but with project.clj modified to use core.matrix 0.55.1-SNAPSHOT (the problem occurs with 0.55.0 and 0.51.0 as well), either with Clojurescript 1.7.228 or 1.9.229. The line numbers given are at the two places where doseq-indexed is used.

WARNING: Wrong number of args (2) passed to clojure.core.matrix.utils/doseq-indexed at line 151 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/x at line 151 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/i at line 151 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/i at line 151 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/x at line 151 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Wrong number of args (2) passed to clojure.core.matrix.utils/doseq-indexed at line 170 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/x at line 170 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/i at line 170 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/i at line 170 resources/public/js/out/clojure/core/matrix/utils.cljc
WARNING: Use of undeclared Var clojure.core.matrix.utils/x at line 170 resources/public/js/out/clojure/core/matrix/utils.cljc

[mikera]

Thanks for the report. We definitely need to get ClojureScript testing integrated, I'm not picking up these things at the moment when I make changes on the clj side.

[mars0i]

For this particular issue, I thought there was a very simple fix: comment out or delete the definition of doseq-indexed in utils.cljc, and pull it in from macros.cljc, where the same definition exists. However, mikera.vectorz.matrix-api expects to find doseq-indexed in clojure.core.matrix.utils. Probably still simple to fix, but I don't know what the preferred strategy would be. Sorry I can't be more helpful with getting testing set up.

[vankasaisumanth]

In the utils.cljc , It is already asked for macros in the line 8 : require macros. So, I would suggest first we need to run macros.cljc first which will give you access to doseq-indexed. Not only doseq-indexed but also someone might have got error at lines 130, 224 where we need TODO and is-long-array as they also need access from macros.cljc . When i run it on online REPL compiler&IDE with version clojure 1.8.0 - 64 bit server um, i got it correct and the code is working. I had run the macros.cljc first followed by utils.cljc . I hope your problem too can be resolved just by running these two in sequence.

[jdkealy]

I can't compile a cljs build using advanced compilations when i include this library because of this line. Any update on this ?