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

Add timit recipe #247

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

Add timit recipe #247

wants to merge 11 commits into from

Conversation

luomingshuang
Copy link
Contributor

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

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

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

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

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

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

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

@csukuangfj csukuangfj csukuangfj requested changes

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