-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
gitignore: add openfst 1.4.1 #1
Conversation
Signed-off-by: Amit Beka <amit.beka@gmail.com>
Github repository is read-only. You should direct the patches directly to me (jtrmal@gmail.com), or to kaldi-developers@sourceforge.net |
@jtrmal: I do not know if it will prove extra work for you, but a PR is looking good, you can apply it in a single command. Just add the .diff extension to any commit or PR URL, and GitHub generates a diff from it. Try https://github.com/kaldi-asr/kaldi/pull/1.diff for this very request. :-) In fact, I just did a couple minutes ago, from a clean sandbox' root wget -qO- http://github.com/kaldi-asr/kaldi/commit/bba05342fa937371a8892eacd162941b83eac42a.diff | patch -p1 |
Interesting, but confusing -- I though I have already applied this patch a while ago. Let me check. |
@jtrmal, I was talking about the process in general, not this particular patch. Sorry I have been unclear. What I meant was you can get the patch right off GitHub instead of directing the user to send it to you, and that is perhaps even easier. |
Oh, I understand now. Thanks for making me realize this -- I saw the links On Wed, Jul 8, 2015 at 6:07 PM, Kirill Katsnelson notifications@github.com
|
As an infrequent contributor to projects I find the github model easy to work with, and I think it will enable more small patches like this to be pushed, if github will be used. |
@amitbeka I am totally with you. Hope kaldi transitions soon : ) On the other hand, the guys are extremely--and I mean extremely extremely busy with the core speech recognition problems, and if you asked me about the priorities, I would rate the github transition somewhat lower. |
Well, @amitbeka, feel free to use the git even now -- there is nothing On Thu, Jul 9, 2015 at 6:52 PM, Kirill Katsnelson notifications@github.com
|
Thanks! On Fri, Jul 10, 2015 at 4:17 AM, jtrmal notifications@github.com wrote:
|
delete unnecessary code from the mikolov rnnlm code; made some stylis…
swap axis for optimization in Tensor3dCopy()
Add ivector support to online nnet3 decoder
change kaldi-asr#2 woooo, let's change the message a bit
Updated and clean up run.sh
[pykaldi] Provide access to the nnet posteriors
Fix build error
paste-vectors and multiply-vectors fix
Signed-off-by: Amit Beka amit.beka@gmail.com