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

nnet3- add binary nnet3-show-progress #62

Closed
danpovey opened this issue Aug 7, 2015 · 2 comments
Closed

nnet3- add binary nnet3-show-progress #62

danpovey opened this issue Aug 7, 2015 · 2 comments
Labels
help wanted Please help us with this issue! nnet3
Milestone

Comments

@danpovey
Copy link
Contributor

danpovey commented Aug 7, 2015

This is for anyone who has the time.
Look at how nnet3-average differs from nnet-am-average to see how you should modify nnet-show-progress to make nnet3-show-progress. It may be necessary to add functions to nnet-utils.h.
The program should read the "raw" nnet format.

@kkm000 kkm000 added this to the nnet3 milestone Aug 18, 2015
@kkm000 kkm000 added help wanted Please help us with this issue! nnet3 labels Aug 18, 2015
@naxingyu
Copy link
Contributor

It requires a ComputeNnetGradient counterpart and I imagine it wouldn't be a nnet-util. I see two option. 1) Update nnet-diagnostics, add a ComputeGradient function, and 2) Add a ComputeGradient function in nnet-compute.h, and the implementation can be copied from nnet-derivative-test.cc. Which one is more of nnet3 spirit? What's your suggestion?

@danpovey
Copy link
Contributor Author

I think what you need already exists in class NnetComputeProb: see the
GetDeriv() function.
Dan

On Sun, Aug 30, 2015 at 9:55 PM, Xingyu Na notifications@github.com wrote:

It requires a ComputeNnetGradient counterpart and I imagine it wouldn't be
a nnet-util. I see two option. 1) Update nnet-diagnostics, add a
ComputeGradient function, and 2) Add a ComputeGradient function in
nnet-compute.h, and the implementation can be copied from
nnet-derivative-test.cc. Which one is more of nnet3 spirit? What's your
suggestion?


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

@danpovey danpovey reopened this Sep 1, 2015
@danpovey danpovey closed this as completed Sep 1, 2015
hainan-xv pushed a commit to hainan-xv/kaldi that referenced this issue Jul 24, 2018
hhadian pushed a commit to hhadian/kaldi that referenced this issue Jan 4, 2019
Add context-extension capability and merged-eg support to nnet3-chain-copy-egs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Please help us with this issue! nnet3
Projects
None yet
Development

No branches or pull requests

3 participants