search

nvaccess / nvda

NVDA, the free and open source Screen Reader for Microsoft Windows
https://www.nvaccess.org/
Other
2.1k stars 634 forks source link

Starting with version 2019.3, the "Capital pitch change" when using the SAPI4 synthesizer was invalid #12311

Closed cary-rowen closed 3 years ago

cary-rowen commented 3 years ago

Below is the debug log I added, You may find helpful information from line 630. nvda_log.txt

Steps to reproduce:

  1. Download and install the speech synthesizer provided in the following link, or use the SAPI4 speech synthesizer already on your computer. link: https://www.dropbox.com/s/3njaa8jrplutk24/IBM_ViaVoice_TTS_Runtime-V6.405.exe?dl=0
  2. Start NVDA and press NVDA + CTRL + S to select "Microsoft Speech API version 4" and confirm.
  3. At this point the NVDA should have switched to the SAPI4 synthesizer and you will need to press NVDA + CTRL + V to open the Voice Setup dialog.
  4. Use the Tab key to navigate until you hear "Capital pitch change percentageeditselected 30" You can change this value to 40 (or higher, or leave it unchanged).
  5. Then use Windows + R to open the Run dialog and type notepad and confirm, this will open notepad allowing you to type some text.
  6. Press the Caps lock key (it may be located to the left of the letter A) You may need to press it twice in a row until you hear "Caps lock on".
  7. Try to enter some letters, such as the letter "C" (or "D" or something else)
  8. Use Left Arrow or Right Arrow to move between these letters.

Actual behavior:

NVDA reports these capital letters with no change in pitch.

Expected behavior:

You will hear NVDA report these capital letters in a higher pitch

System configuration

NVDA installed/portable/running from source:

installed and portable.

NVDA version:

NVDA 2020.4

Windows version:

Windows 10 20H2 (64Bit) build 19042.928

Name and version of other software in use when reproducing the issue:

None

Other information about your system:

None

Other questions

Does the issue still occur after restarting your computer?

Yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

I tested NVDA2020.4, NVDA2020.3, NVDA2019.3 And NVDA2019.2.1 in turn and found that this is a regression introduced by NVDA 2019.3(Of course including 2019.3.1).

If add-ons are disabled, is your problem still occurring?

Yes.

Does the issue still occur after you run the COM Registration Fixing Tool in NVDA's tools menu?

Yes.

vgjh2005 commented 3 years ago

Hi: I agree with you. It is a Critical issue! SAPI version 4 is still the most important synthesizer today. Though there are more and more modern complex synthesizer switching system plugins, but using different sounds to announce different languages do to text reading is not enough of satisfied. Fortunately, IBM viavoice text to speech engine has been able to run to announce multi language at the same time by one voice. It is still my only synthesizer. So please fix it. Thanks!

zstanecic commented 3 years ago

Hi,

The workaround for this is to turn off use spelling functionality checkbox in the NVDA voice settings.

When this setting is turned off, the speech returns to normal, i.e capital pitch change is normal.

From: vgjh2005 @.> Sent: Wednesday, April 21, 2021 10:36 AM To: nvaccess/nvda @.> Cc: Subscribed @.***> Subject: Re: [nvaccess/nvda] Starting with version 2019.3, the "Capital pitch change" when using the SAPI4 synthesizer was invalid (#12311)

Hi: I agree with you. It is a Critical issue! SAPI version 4 is still the most important synthesizer today. Though there are more and more modern complex synthesizer switching system plugins, but using different sounds to announce different languages do to text reading is not enough of satisfied. Fortunately, IBM viavoice text to speech engine has been able to run to announce multi language at the same time by one voice. It is still my only synthesizer. So please fix it. Thanks!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nvaccess/nvda/issues/12311#issuecomment-823886737 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVCDE4MJLKQYEZL2WSSBR3TJ2FAVANCNFSM43H2Q3EA . https://github.com/notifications/beacon/ACVCDE23FPJLFU5QPXXT7PTTJ2FAVA5CNFSM43H2Q3EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGENYHEI.gif

cary-rowen commented 3 years ago

Thanks, I have tried this method and it does not work as expected, so it is invalid. @zstanecic

cary-rowen commented 3 years ago

Can you investigate this? Thank you. @CyrilleB79

zstanecic commented 3 years ago

Ach, this works for one type of sapi4 synths, but not for others.

From: Rowen @.> Sent: Wednesday, April 21, 2021 11:16 AM To: nvaccess/nvda @.> Cc: zstanecic @.>; Mention @.> Subject: Re: [nvaccess/nvda] Starting with version 2019.3, the "Capital pitch change" when using the SAPI4 synthesizer was invalid (#12311)

Thanks, I have tried this method and it does not work as expected, so it is invalid. @zstanecic https://github.com/zstanecic

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nvaccess/nvda/issues/12311#issuecomment-823911910 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVCDE4M7RZZWQI2ZLVPY5LTJ2JVZANCNFSM43H2Q3EA . https://github.com/notifications/beacon/ACVCDE5573H2XSUHVTBBCKTTJ2JVZA5CNFSM43H2Q3EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGEN6LZQ.gif

cary-rowen commented 3 years ago

Yes, so I believe this is a bug of NVDA. In the synthesizer download link I provided, this method is invalid.

zstanecic commented 3 years ago

Aare there any debug logs?

From: Rowen @.> Sent: Wednesday, April 21, 2021 11:28 AM To: nvaccess/nvda @.> Cc: zstanecic @.>; Mention @.> Subject: Re: [nvaccess/nvda] Starting with version 2019.3, the "Capital pitch change" when using the SAPI4 synthesizer was invalid (#12311)

Yes, so I believe this is a bug of NVDA. In the synthesizer download link I provided, this method is invalid.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nvaccess/nvda/issues/12311#issuecomment-823919830 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVCDE5INEFATNVTZMY2LWDTJ2LD3ANCNFSM43H2Q3EA . https://github.com/notifications/beacon/ACVCDEZG6N7IAHK652Y6KFDTJ2LD3A5CNFSM43H2Q3EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGEOAJVQ.gif

cary-rowen commented 3 years ago

@zstanecic I have added the debug log at the very beginning of the first comment. Thank you for your reminder and hope you can assist in the investigation.

zstanecic commented 3 years ago

The thing is very simple, but it should be rewritten with people who knows sapi4.

Pitch command is not supported at all, but it should be.

From: Rowen @.> Sent: Wednesday, April 21, 2021 1:59 PM To: nvaccess/nvda @.> Cc: zstanecic @.>; Mention @.> Subject: Re: [nvaccess/nvda] Starting with version 2019.3, the "Capital pitch change" when using the SAPI4 synthesizer was invalid (#12311)

@zstanecic https://github.com/zstanecic I have added the debug log at the very beginning of the first comment. Thank you for your reminder and hope you can assist in the investigation.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nvaccess/nvda/issues/12311#issuecomment-824004306 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVCDE2VM44FYMF7QMVOCQTTJ24Z3ANCNFSM43H2Q3EA . https://github.com/notifications/beacon/ACVCDE6DHHRQDRYJGTMN3SLTJ24Z3A5CNFSM43H2Q3EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGEOU5UQ.gif

dpy013 commented 3 years ago

cc @seanbudd