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

why rnnlm tool doesn't have parameter for syn0 --> hidden #97

Closed
fedorajzf opened this issue Aug 27, 2015 · 6 comments
Closed

why rnnlm tool doesn't have parameter for syn0 --> hidden #97

fedorajzf opened this issue Aug 27, 2015 · 6 comments

Comments

@fedorajzf
Copy link

No description provided.

@danpovey
Copy link
Contributor

Ilya edrenkin wrote the RNNLM tool. I don't understand what the question
means.
Ilya, if you could get a github account it would be helpful.
Dan

On Thu, Aug 27, 2015 at 8:41 AM, fedorajzf notifications@github.com wrote:


Reply to this email directly or view it on GitHub
#97.

@ilya-eder
Copy link

Here I am.
However I also don't get the question, frankly. syn0 matrix is used for word embeddings, they are just learned. What do you mean by a parameter in this context?

@danpovey
Copy link
Contributor

This is more of a discussion than a real issue; closing the issue.

@fedorajzf
Copy link
Author

hi, ilya
in rnnlm.c,hidden value is stored in neu1, neu1[t] = sigmoid(synRec * neu1[t-1] + syn0)
should it be neu1[t] = sigmoid(synRec * neu1[t-1] + syn0 * U) ? here U is another parameter for syn0 to hidden layer.

@ilya-eder
Copy link

It might be the case that carefully choosing this U would improve convergence; however I'd expect from the training procedure to learn the optimal scale for syn0 without making it a parameter.

@fedorajzf
Copy link
Author

if not use U, hidden layer dimension must equal embedding dimension, maybe this is a problem

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

3 participants