Skip to content

Some update to the LSTM code.#63

Merged
carriepl merged 27 commits into
lisa-lab:masterfrom
nouiz:master
Jan 14, 2015
Merged

Some update to the LSTM code.#63
carriepl merged 27 commits into
lisa-lab:masterfrom
nouiz:master

Conversation

@nouiz
Copy link
Copy Markdown
Member

@nouiz nouiz commented Jan 8, 2015

No description provided.

Comment thread code/lstm.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should a comment be here or was the '#' a typo?

@nouiz
Copy link
Copy Markdown
Member Author

nouiz commented Jan 9, 2015

Can someone review this PR and merge it? I think it do a good code clean up and updates.

Comment thread code/lstm.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"minutes" => "seconds"?

@carriepl
Copy link
Copy Markdown
Contributor

Ok, this looks good enough for merge. Good work.

carriepl added a commit that referenced this pull request Jan 14, 2015
Some update to the LSTM code.
@carriepl carriepl merged commit b7481db into lisa-lab:master Jan 14, 2015
taneishi pushed a commit to taneishi/DBN that referenced this pull request Nov 28, 2019
Some update to the LSTM code.
taneishi pushed a commit to taneishi/DBN that referenced this pull request Feb 13, 2020
Some update to the LSTM code.

Former-commit-id: 8165e0c
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

Successfully merging this pull request may close these issues.

2 participants