Skip to content
This repository was archived by the owner on Oct 13, 2022. It is now read-only.

Add timit recipe #247

Open
luomingshuang wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add timit recipe #247

luomingshuang wants to merge 11 commits into from

Conversation

@luomingshuang
Copy link
Contributor

@luomingshuang luomingshuang commented Sep 3, 2021

I set subsampling_factor=2.
#TIMIT CTC_Train
Testing results based on different training epochs:

epoch=20
2021-09-03 10:54:10,903 INFO [ctc_decode.py:188] %PER 30.34% [2225 / 7333, 293 ins, 441 del, 1491 sub ]

epoch=30
2021-09-03 10:59:10,147 INFO [ctc_decode.py:188] %PER 29.77% [2183 / 7333, 221 ins, 473 del, 1489 sub ]

epoch=35
2021-09-03 11:11:00,885 INFO [ctc_decode.py:188] %PER 28.94% [2122 / 7333, 266 ins, 397 del, 1459 sub ]

epoch=40
2021-09-03 11:12:39,029 INFO [ctc_decode.py:188] %PER 29.52% [2165 / 7333, 304 ins, 348 del, 1513 sub ]

@csukuangfj
Copy link
Collaborator

csukuangfj commented Sep 3, 2021

https://github.com/speechbrain/speechbrain/tree/develop/recipes/TIMIT/ASR/CTC

The PER in speechbrain is about 14.78. Seems there is a lot of room for improvement

@luomingshuang
Copy link
Contributor Author

luomingshuang commented Sep 3, 2021

Yes, I am having a experiment with mmi, and I will have a look at their codes.

https://github.com/speechbrain/speechbrain/tree/develop/recipes/TIMIT/ASR/CTC

The PER in speechbrain is about 14.78. Seems there is a lot of room for improvement

@luomingshuang
Options for the number of phones
d892db7 
Choose a number (60, 48, 29) of phones for modeling
csukuangfj
csukuangfj suggested changes Sep 12, 2021
View reviewed changes
egs/timit/asr/simple_v1/local2/train_lms.sh Outdated
done

# This script takes no arguments. It assumes you have already run
# aishell_data_prep.sh.
Copy link
Collaborator

@csukuangfj csukuangfj Sep 12, 2021

Choose a reason for hiding this comment

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

aishell or TIMIT?

Copy link
Contributor Author

@luomingshuang luomingshuang Sep 12, 2021

Choose a reason for hiding this comment

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

TIMIT. Have fixed.

egs/timit/asr/simple_v1/path.sh Outdated
@@ -0,0 +1,3 @@
export SNOWFALL_ROOT=`pwd`/../../../..
[ -f $SNOWFALL_ROOT/tools/env.sh ] && . $SNOWFALL_ROOT/tools/env.sh
Copy link
Collaborator

@csukuangfj csukuangfj Sep 12, 2021

Choose a reason for hiding this comment

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

What does env.sh contain?

Copy link
Contributor Author

@luomingshuang luomingshuang Sep 12, 2021

Choose a reason for hiding this comment

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

Em...It (path.sh) is a unnecessary file. Have deleted.

@luomingshuang
Update train_lms.sh
5ee228f
@luomingshuang
Update run.sh
44e7157
@luomingshuang
Delete path.sh
8bed123
@luomingshuang
Add CRDNN model for timit recipe
07b9bdb 
Add CRDNN model for timit recipe. The results get better based on this model than TdnnLstm1b.
@luomingshuang
Rename
54e4476
@luomingshuang luomingshuang mentioned this pull request Oct 28, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@csukuangfj csukuangfj csukuangfj requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@luomingshuang @csukuangfj