The Wayback Machine - https://web.archive.org/web/20200711064221/https://github.com/eclipse/deeplearning4j/issues/8634
Skip to content

/ deeplearning4j

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

Jump to bottom

Non-deterministic Word2Vec if Seed is 0 #8634

Open
orausch opened this issue Jan 23, 2020 · 3 comments
Open

Non-deterministic Word2Vec if Seed is 0 #8634

orausch opened this issue Jan 23, 2020 · 3 comments
Labels
Documentation

Comments

@orausch
Copy link

@orausch orausch commented Jan 23, 2020 •
edited

Issue Description

The following test fails for seed = 0 but passes for (as far as I can tell) any other seed (e.g. seed = 1)

https://gist.github.com/orausch/9a42e24b782319447a515e8c29b364a0

Version Information

Please indicate relevant versions, including, if relevant:

  • Deeplearning4j version: beta6
  • Platform information (OS, etc): Ubuntu 19.10
    (cc @rpatra)
@raver119
Copy link
Contributor

@raver119 raver119 commented Jan 23, 2020

Seed 0 means "no seed", and effectively is the same as using System.currentTimeMillis() as seed. So this is intentional behavior.
@raver119 raver119 closed this Jan 23, 2020
@orausch
Copy link
Author

@orausch orausch commented Jan 23, 2020

Ah okay, I was thinking that it was something among those lines. Does this also hold for Nd4j?

It may be helpful to update the docs, since this isn't really common from other ML frameworks. While debugging I looked here and didn't find anything:
https://deeplearning4j.org/api/latest/org/nd4j/linalg/api/rng/Random.html#setSeed-int-
https://deeplearning4j.org/docs/latest/nd4j-overview#createrandom
@raver119
Copy link
Contributor

@raver119 raver119 commented Jan 23, 2020

Mmmm, good point, thank you
@raver119 raver119 reopened this Jan 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@raver119 @orausch
You can’t perform that action at this time.