Opened 2 years ago

Closed 7 months ago

Last modified 7 months ago

#3184 closed task (fixed)

Remove Newfon synth from NVDA

Reported by: jteh Owned by: James Teh <jamie@nvaccess.org>
Milestone: 2015.1 Component: Speech synth drivers
Keywords: Cc: aleksey_s, zstanecic
Blocked by: Blocking: #4696

Description

#206 added the Newfon driver to NVDA. The decision was controversial even back then, as Newfon itself can't be bundled with NVDA even if we wanted to. Newfon requires users to install files into the NVDA program directory, which we have always advised against.

Now that the add-on framework exists, I think it makes far more sense for Newfon to be an add-on. Among other things, this will make it very easy to install.

Change History (14)

comment:1 Changed 2 years ago by jteh

I just found out Tyler Spivey converted this into an add-on a while ago: https://dl.dropboxusercontent.com/u/1060978/newfon.nvda-addon?dl

comment:2 Changed 2 years ago by aleksey_s

I do not mind.

About community addons - users have been writing to me asking for support of different non-working versions of addons packaging newfon, so there should be a number of those. If we remove this from the core, it will hopefuly force me to release the official one.

comment:3 follow-up: Changed 2 years ago by jteh

While I have your attention :), I have a patch here which forces the encoding to cp1251, as Newfon requires this and the driver currently uses the system's default ANSI encoding. I think I mentioned this to you a while back, but I forget your response.

How would you like to proceed with this? Did you want help from us with finishing the add-on before we remove it?

comment:4 in reply to: ↑ 3 Changed 2 years ago by aleksey_s

Replying to jteh:

While I have your attention :), I have a patch here which forces the encoding to cp1251, as Newfon requires this and the driver currently uses the system's default ANSI encoding. I think I mentioned this to you a while back, but I forget your response.

Patch is welcome. It is one of things which I always knew should be fixed but didn't because they had not anoyed me directly :-)

How would you like to proceed with this? Did you want help from us with finishing the add-on before we remove it?

Yes. I'll be able to work on this in a week, after I finish my course paper.

comment:5 Changed 9 months ago by nvdakor

Hi,
Isn't there a more recent ticket requesting the same thing? Someone said the Russian NVDA community site is down and asked that this synth be moved out of the core.
Thanks.

comment:6 Changed 9 months ago by jteh

  • Blocking 4696 added

comment:7 Changed 9 months ago by jteh

  • Cc zstanecic added
  • Summary changed from Remove Newfon synth from NVDA and make it an add-on to Remove Newfon synth from NVDA

I think we should just go ahead and remove this. It's true that the add-on isn't currently easily available, but given that the Russian community site has been down for some time (see #4696), this isn't readily available now anyway. Ideally, we'd want to get the add-on included on addons.nvda-project.org, but for now, I guess we can at least update the ExtraVoices page.

comment:8 Changed 9 months ago by jteh

On a related note, I've now removed the Russian community site from the Connect page and from nvda-community.org.

comment:9 Changed 8 months ago by zstanecic

the addon works well, tyler's addon is 100 % working.
i can use it on any system on 1251cp.

comment:10 Changed 7 months ago by James Teh <jamie@nvaccess.org>

  • Owner set to James Teh <jamie@nvaccess.org>
  • Resolution set to fixed
  • Status changed from new to closed

In 708d6efe3e30d5d3c71381010d1338836715ac6c:

Support for the Newfon synthesizer has been removed. Newfon is now available as an NVDA add-on.

Fixes #3184.

comment:11 Changed 7 months ago by jteh

  • Milestone changed from near-term to 2015.1

comment:12 Changed 7 months ago by bdorer

well, I don't know wheather some users are using this addon, but If s.o. reads the addon is publicly available s.o. should update wiki with the link. Ideally it should be released from addons community page as other people now they're downloading tested addons.

comment:13 Changed 7 months ago by jteh

The ExtraVoices page has already been updated with the link.

I'd like to list it on addons.nvda-project.org, but the requirements for getting something listed there are currently far too stringent IMO. In short, neither Tyler nor NV Access want to (or can) maintain this add-on; it is just a service to the community. However, it being available and unmaintained is better than it not being available at all. This is something that Mesar, myself and others need to discuss further.

comment:14 Changed 7 months ago by nvdakor

Hi,
Sent a proposal to have add-ons community site host the add-on version. Thanks.

Note: See TracTickets for help on using tickets.