dselivanov / text2vec

Fast vectorization, topic modeling, distances and GloVe word embeddings in R.
http://text2vec.org
Other
848 stars 135 forks source link

GlobalVectors object lost fit method? #329

Open geotheory opened 3 years ago

geotheory commented 3 years ago

Hey there. I find your example from Feb no longer works for me (macOS 11.0.1, version 4.0.3, text2vec 0.6).

require(text2vec)
#> Loading required package: text2vec
glove = GlobalVectors$new(rank = 50, x_max = 10)
glove$fit
#> NULL

In your example script, replacing glove$fit() line with glove$fit_transform() does train the model. But next line fails:

word_vectors <- glove$get_word_vectors()
Error: attempt to apply non-function

because glove$get_word_vectors is also missing. Any idea what is going on?

dselivanov commented 3 years ago

Thanks for reporting, I will take a look.

On Sat, 26 Dec 2020, 19:49 Robin Edwards, notifications@github.com wrote:

Hey there. Is this package still maintained?

I find your example https://cran.r-project.org/web/packages/text2vec/vignettes/glove.html from Feb no longer works for me (macOS 11.0.1, version 4.0.3, text2vec 0.6).

require(text2vec)

> Loading required package: text2vec

glove = GlobalVectors$new(rank = 50, x_max = 10) glove$fit

> NULL

In your example script https://cran.r-project.org/web/packages/text2vec/vignettes/glove.html, replacing glove$fit() line with glove$fit_transform() does train the model. But next line fails:

word_vectors <- glove$get_word_vectors() Error: attempt to apply non-function

because glove$get_word_vectors is also missing. Any idea what is going on?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dselivanov/text2vec/issues/329, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHC5XKE4KCNTJVZGUSONFDSWYAYLANCNFSM4VKAFRGA .

geotheory commented 3 years ago

I just noticed the vignette has changed since that example. April version is up to date.

dselivanov commented 3 years ago

Yes, I need to update CRAN version to fix some documentation issues. This issue seems duplicate for #323. I will keep it open until I update pkg on CRAN.