-
-
Notifications
You must be signed in to change notification settings - Fork 783
Remove MathPlayer #19165
Copy link
Copy link
Labels
api-breaking-changeaudience/nvda-devPR or issue is relevant to NVDA / Add-on developersPR or issue is relevant to NVDA / Add-on developersp3https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityhttps://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityrelease/blockingthis issue blocks the milestone releasethis issue blocks the milestone releaserelease/blocking-betatriagedHas been triaged, issue is waiting for implementation.Has been triaged, issue is waiting for implementation.
Milestone
Description
Metadata
Metadata
Assignees
Labels
api-breaking-changeaudience/nvda-devPR or issue is relevant to NVDA / Add-on developersPR or issue is relevant to NVDA / Add-on developersp3https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityhttps://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priorityrelease/blockingthis issue blocks the milestone releasethis issue blocks the milestone releaserelease/blocking-betatriagedHas been triaged, issue is waiting for implementation.Has been triaged, issue is waiting for implementation.
Type
Fields
Give feedbackNo fields configured for Task.
Related issues, PRs or discussions
Blocked by #18323
What is the current state of the codebase?
MathPlayer is a legacy and unmaintained form of reading Math with NVDA. MathPlayer is broken with 64bit NVDA. We are replacing it with MathCAT.
Why are changes required?
There is no native math support with NVDA and the current code for (external) MathPlayer has been replaced by the MathCAT add-on in usage and recommendations. We are integrating the MathCAT add-on and hence MathPlayer should be removed.
What technical changes are required?
Removing math player code from NVDA python,
miscDeps\include\mathPlayerand documentationAre the proposed technical changes API breaking?
Yes
Are there potential risks or issues with the proposed implementation?
MathCAT requires testing and may require further work