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
Merge the synthesizer settings into the Voice settings. #4040
Comments
Comment 1 by briang1 on 2014-03-29 07:55 |
Comment 2 by jteh (in reply to comment description) on 2014-03-29 10:00
I don't follow this. If the synth isn't present or has some other error, you will always get an error when switching to it. Even if we did this, we wouldn't change the synth as soon as you select it in the list. |
Comment 3 by vgjh2005 (in reply to comment 1) on 2014-03-30 11:35
IBM developed two synthesizers , the first called IBM chinese text to speech,and the second one called ViaVoice text to speech. Press NVDa+CTRL+S to select the interface microsoft sapi version 4.0. They are developed on the interface. When I selected |
Comment 4 by jteh on 2014-03-30 20:48 |
Comment 5 by vgjh2005 (in reply to comment 4) on 2014-04-01 11:11
I opened the NVDA log viewer when loading is failure. But I could not get the error log. Following is the download link of the synthesizer IBM chstts: http://hot.baidupcs.com/file/159d9d33aa8e92c4031b9981a7ecfd29?xcode=b79646ed9bc35b7df124c695489b987836728e07a6c7d4b6&fid=671288931-250528-1467621449&time=1396350438&sign=FDTAXER-DCb740ccc5511e5e8fedcff06b081203-Cvao4tqcmQv3OAEXgJuQ%2BIcQggo%3D&to=hb&fm=N,B,U,nc&newver=1&expires=1396351038&rt=sh&r=469535456&logid=528681724&sh=1&vuk=282335&fn=chstts.rar |
@jcsteh Could you please summarize your arguments against the proposed design change? While the proposal does make sense to me, I do understand how refreshing all voice parameters when the a different synthesizer is selected could cause performance problems. |
I think this request will be redundant as soon as #577 is done. Still, synthesizer and voice settings will be two separate categories in the new settings dialog, but it is much easier to go from panel to panel, i.e. it doesn't require closing and opening dialogs. Having said that, leaning towards a won't fix. |
I think this will be academic when the tab based property sheets are put
into the snapshots as tabbing is quite easy, and even now the short cut
keys are faster than the menu if you are a bit pushed for time.
Brian
bglists@blueyonder.co.uk
Sent via blueyonder.
Please address personal email to:-
briang1@blueyonder.co.uk, putting 'Brian Gaff'
in the display name field.
|
While #577 perhaps makes this less necessary, it's worth noting that switching categories probably doesn't apply settings (or does it?). If it doesn't, that will actually cause problems for voice settings if the synthesizer is changed. |
@jcsteh commented on 14 aug. 2017 08:49 CEST:
Nope, it doesn't, you will explicitly have to click apply.
It wil indeed show the voices for the old synthesizer, unless you have explicitly clicked apply, what you probably want to do anyway if you'd like to test the new synthesizer. But still, that's something that needs to be documented for #577, or we should popup a question when one tries to leave the synthesizer dialog without having the new synth applied. |
I suppose I agree with the reasoning provided by @jcsteh in #4040 (comment). Also,In particular, #577 will also likely make this ticket less necessary. All in all, may we close this ticket as a wontfix? @ehollig |
Closing as wontfix because of reasoning expressed above. Also, #577 may, or may not make this issue less necessary. |
Reported by vgjh2005 on 2014-03-29 07:35
Hi:
I think the voice setting and synthesizer can merge to be one. The reason is below:
1.If the selected synthesizer interface contain a synthesizer that is the default one and NVDA cannot support it, it will not cause error. Because the synthesizer settings mustn't be refreshed.
2. It can be frugal of a hotkey.
3. It looks like more logicaler.
4. It can be frugle of user's time.
After merge them, add a checkbox to define if the voice should be refreshed realtime is more better.
Thanks.
The text was updated successfully, but these errors were encountered: