Maciej Kula

Yes, it's far from clear. I'll have a think and see if I can come up with something. On 23 Aug 2016 21:52, "Anthony Nowell" [email protected] wrote: > I'll be...

Can you try to find out which file it thinks it's missing?

OK. I'll try to reproduce when I get a bit of time.

A couple of tips: 1. You need to install the package before you can run the code. This is normally done by running `pip install glove_python` or `pip install .`...

Makes sense to track the loss. A couple of comments: 1. Why not just use a primitive `double` that's initialized in Cython, and then maybe returned from the `fit_vectors` function?...

It just seems strange to have a one-element array when the same purpose can be served by just having a single number. Regarding the multithreading issue: I don't think this...

I was going to go in sneakily after this is merged and do that :) @ducovrossem as Radim pointed out, it will be more efficient to only calculate the `(prediction...

The raw value is used for the weighting (and I also quite like the fact that the co-occurrence matrix is marginally model agnostic and could conceivably support a different application)....

Looks good at first glance. I'll merge when I have access to a computer. On 10 Dec 2014 00:02, "Ossama W. Obeid" [email protected] wrote: > Added a distance method to...

A couple of comments: 1. Could you encapsulate checking whether the model has been fitted into its own method so that we could reuse it in both your new method...