Lstm noprojection

[naxingyu]

Add LSTM component without projection.

[danpovey]

Why are you putting the LSTM component stuff in the same pull request as your Maxout test code? For sure this will delay the merging of the code. Can we do this one issue at a time?

[naxingyu]

Thanks Dan. Sorry for that. I thought github could distinguish pull requests from different commits... I close this request now.

[jtrmal]

It will... Buf you have to create the PRs from different branches. Then it
will work as expected.
y.

On Fri, Jul 24, 2015 at 7:41 PM, Xingyu Na notifications@github.com wrote:

Closed #23 #23.

—
Reply to this email directly or view it on GitHub
#23 (comment).

[naxingyu]

Yes, I created a new branch, but the old commits are also in this new branch... I must have done something wrong, will figure it out.

[jtrmal]

I see... no, the unrelated commits have to be in separate branch :)
you can use git cherry-pick to pick the individual commits
http://stackoverflow.com/questions/25955822/git-cherry-pick-a-single-commit-for-pull-request

On Fri, Jul 24, 2015 at 7:48 PM, Xingyu Na notifications@github.com wrote:

Yes, I created a new branch, but the old commits are also in this new
branch... I must have done something wrong, will figure it out.

—
Reply to this email directly or view it on GitHub
#23 (comment).

[danpovey]

or just wait till the first thing is committed.

On Fri, Jul 24, 2015 at 4:51 PM, jtrmal notifications@github.com wrote:

I see... no, the unrelated commits have to be in separate branch :)
you can use git cherry-pick to pick the individual commits

http://stackoverflow.com/questions/25955822/git-cherry-pick-a-single-commit-for-pull-request

On Fri, Jul 24, 2015 at 7:48 PM, Xingyu Na notifications@github.com
wrote:

Yes, I created a new branch, but the old commits are also in this new
branch... I must have done something wrong, will figure it out.

—
Reply to this email directly or view it on GitHub
#23 (comment).

—
Reply to this email directly or view it on GitHub
#23 (comment).