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

Merged 'master' branch #3127

Merged
merged 28 commits into from
Mar 17, 2019
Merged

Merged 'master' branch #3127

merged 28 commits into from
Mar 17, 2019

Conversation

desh2608
Copy link
Contributor

No description provided.

desh2608 and others added 28 commits March 10, 2019 14:13
clang is unhappy with '-rdynamic' in compile-only step, and the
switch is really unnecessary.

Also, the default location for MKL 64-bit libraries is intel64/.
The em64t/ was explained already obsolete by an Intel rep in 2010:
https://software.intel.com/en-us/forums/intel-math-kernel-library/topic/285973
The type of --max-deleted-words-kept-when-merging in segment_ctm_edits.py
was a string, which prevented the mechanism from working altogether.
@danpovey danpovey merged commit 2a69516 into kaldi-asr:kaldi10 Mar 17, 2019
danpovey added a commit that referenced this pull request Mar 17, 2019
@danpovey
Copy link
Contributor

@desh2608 could you please make this PR under another name with the same source branch?
To keep the history clean (and ensure we merge only with actual merge) I used git push -f to remove this merge from the history, but it has confused GitHub.

@desh2608 desh2608 deleted the kaldi10 branch March 26, 2019 19:21
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.