Skip to content
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

Major Concern about multi-data- approach to SSML encoding #86

Open
paulgrudnitski opened this issue May 27, 2021 · 1 comment
Open

Major Concern about multi-data- approach to SSML encoding #86

paulgrudnitski opened this issue May 27, 2021 · 1 comment
Labels
AT Assistive Technologies

Comments

@paulgrudnitski
Copy link

It's exciting to see this specification reach this stage. Congratulations to the many contributors.

One aspect of this multiple vs. single attribute debate that I have not seen discussed is "performance" - as in run-time performance - when implementing either approach. After having implemented this draft specification (the single-attribute approach) one thing that I can state for certain is that sniffing/selecting for up to 29 different data-ssml-* attributes on every html element/tag in the DOM is going to be 20+ times worse than sniffing/selecting 1 data-ssml attribute and unpacking that json to tease out the ssml information via a json processor.

From experience, when a TTS player has to rip through the DOM in real time, scanning each element for all of data-ssml-* attributes that the TTS player supports (up to 29!?) before sending the information off to the device's (or player's) speech synthesis API ... well that's going to be a brutal experience full of unwanted pauses unless you are running on some amazing equipment which doesn't exist in the K-12 space where I've done my implementation.

Just my $0.02 cents: don't let style win over substance.

@AutoSponge
Copy link
Contributor

Thank you for the comment. It's still being reviewed by the TF. We will pay attention to performance as implementations are presented.

@alia11y alia11y added the AT Assistive Technologies label Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT Assistive Technologies
Projects
None yet
Development

No branches or pull requests

3 participants