glint-fm icon indicating copy to clipboard operation
glint-fm copied to clipboard

Compilation doesn't work due to forward reference of variable

Open darkseed opened this issue 9 years ago • 3 comments
trafficstars

I've tried to compile your code, but it fails due to a forward reference of k

[info] Compiling 4 Scala sources to /Users/muldert/Sources/glint-fm/target/scala-2.10/classes...
[error] .../glint-fm/src/main/scala/com/github/mlnick/glintfm/GlintFM.scala:187: forward reference extends over definition of value x$7
[error]             val randV = BDM.rand(localVDim, k, Rand.gaussian(0.0, initStd))
[error]                                             ^
[error] one error found
[error] (compile:compileIncremental) Compilation failed

darkseed avatar Nov 07 '16 08:11 darkseed

Ah sorry about that - it was a rush to push it out in time for Spark Summit talk. Should be fixed in https://github.com/MLnick/glint-fm/commit/19de327879b8d2645b60436ac93afd8b16f62dec

MLnick avatar Nov 07 '16 09:11 MLnick

By the way, the code here is really very rough and more a PoC than anything near production ready :)

MLnick avatar Nov 07 '16 09:11 MLnick

Very rough is good enough for now, just wanted to look at your code, after viewing your presentation you gave at Spark Summit. Nice response time on the reported issue ;)

darkseed avatar Nov 07 '16 09:11 darkseed