search

thunderdrop / IBMTTSDictionaries

A large, community-driven pronunciation dictionary for the IBMTTS speech synthesizer in American English
Creative Commons Zero v1.0 Universal
22 stars 9 forks source link

2020-09 - main #11

Closed ultrasound1372 closed 4 years ago

ultrasound1372 commented 4 years ago

See readme for contributing guidelines. Any words you wish added, propose them here.

ultrasound1372 commented 4 years ago

Due to lack of entries in this category, instead of duplicating a new issue with the same text and closing this one, I will rename this issue for the new month.

ultrasound1372 commented 4 years ago

@amirsol81 I do not agree with the addition of "A" as an entry in ENUmain. What purpose does this serve? It just mangles some pronunciations for me.

amirsol81 commented 4 years ago

@ultrasound1372 I mentioned the rationale for the inclusion of capital a in its commit: headlines containing capital a are read erroneously. Take these two, for instance:

Hero Rat Wins A Top Animal Award For Sniffing Out Landmines Kim Jong Un Says He's Sorry That North Korean Troops Killed A South Korean Man

So may I know where this addition messes up with Eloquence? We can remove it if needed -- again as mentioned in the commit.

On 9/25/2020 8:47 PM, Colton Hill wrote:

@amirsol81 https://github.com/amirsol81 I do not agree with the addition of "A" as an entry in ENUmain. What purpose does this serve? It just mangles some pronunciations for me.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/thunderdrop/IBMTTSDictionaries/issues/11#issuecomment-699050258, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIGLZMTFEXWV2KXSKMPOD3TSHTGCLANCNFSM4PR4CKBA.

ultrasound1372 commented 4 years ago

Since it's the 1st in UTC, I'm closing this to prepare the release and open new issues for next month.

ultrasound1372 commented 4 years ago

@amirsol81 you can use this regex in your NVDA voice dictionary to make capital A be properly read if encountered in the context of a headline.

pattern: (\s+)A(\s+)
replacement: \1a\2
case sensitive checked
type Regular expression
amirsol81 commented 4 years ago

@ultrasound1372 Thanks for the tip. I'll make use of it.

ultrasound1372 commented 4 years ago

@amirsol81 actually, don't. There's a nasty flaw in that expression as it stands, it will lowercase the "A" at the beginning of a sentence, thus breaking punctuation parsing rules. A more suitable pattern might be the following.

([^.?!:;]\s+)A(\s+)