Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Method not defined on using LDA #208

Closed
craigpfeifer opened this issue Sep 15, 2014 · 1 comment
Closed

Method not defined on using LDA #208

craigpfeifer opened this issue Sep 15, 2014 · 1 comment

Comments

@craigpfeifer
Copy link

In kicking the tires on LDA with my test text set, i get the following error:

Exception in thread "main" java.lang.Error: Method zero() not defined on class cc.factorie.variable.SortedSparseCountsProportions1
at cc.factorie.la.ReadOnlyTensor$class.zero(Tensor.scala:145)
at cc.factorie.variable.SortedSparseCountsProportions1.zero(ProportionsVariable.scala:359)
at cc.factorie.app.topics.lda.SparseLDAInferencer$$anonfun$exportThetas$1.apply(SparseLDAInferencer.scala:105)
at cc.factorie.app.topics.lda.SparseLDAInferencer$$anonfun$exportThetas$1.apply(SparseLDAInferencer.scala:103)
at scala.collection.Iterator$class.foreach(Iterator.scala:727)
at scala.collection.AbstractIterator.foreach(Iterator.scala:1157)
at scala.collection.MapLike$DefaultValuesIterable.foreach(MapLike.scala:206)
at cc.factorie.app.topics.lda.SparseLDAInferencer.exportThetas(SparseLDAInferencer.scala:103)
at cc.factorie.app.topics.lda.LDA.inferTopics(LDA.scala:146)
at cc.factorie.app.topics.lda.LDACmd.main(LDA.scala:426)
at cc.factorie.app.topics.lda.LDA.main(LDA.scala)

this is on a fresh clone of the project, and a fresh build from source. My command line invocation is:

./factorie/bin/fac lda --read-lines=./data/courts/scotus/cleaned/all_cleaned.txt --num-topics 200 --print-topics-phrases=10

It also fails with the sample command line provided ;

bin/fac lda --read-dirs mytextdir --num-topics 20 --num-iterations 100

And I'm happy to provide the data if it is of interest.

Perhaps something I'm missing?

strubell added a commit that referenced this issue Oct 19, 2014
fix issue #208: Method not defined on using LDA
@strubell
Copy link
Member

Resolved in PR #221

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants