Skip to content

Conversation

@vaaale
Copy link

@vaaale vaaale commented Mar 2, 2016

To accommodate larger vocabs

@markhull
Copy link

Just to throw my 2 cents worth in, I'm using the result of a big run which has vocab size = 4,889,031 and layer size = 500, the product of which is still > MAXINT. This will exceed your new buffer calculation as well and still cause a Negative Capacity error, if I'm reading the docs correctly.

@dirkgr
Copy link
Contributor

dirkgr commented Mar 21, 2016

Why does this have so many changes? Did you change the indentation of that file?

@vaaale
Copy link
Author

vaaale commented Mar 24, 2016

Hi.

Didn't mean to do so, but IntelliJ might have done it automatically. The
only change I made was to the buffer allocation to accommodate lager models.

Best

Alex

On Mon, Mar 21, 2016 at 11:07 PM, Dirk Groeneveld [email protected]
wrote:

Why does this have so many changes? Did you change the indentation of that
file?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#37 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants