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

hotfix:natural speech windows-1252 process stream crash #6097

Merged
merged 3 commits into from
Jun 4, 2024

Conversation

TheLouisHong
Copy link
Contributor

RuneLite JVM launches with -Dfile.encoding=windows-1252

We're unsure when/if this change occurred, but Natural Speech now crashes piper text-to-speech engine when writing to standard stream with windows-1252 encoding. Natural Speech no longer works!

The changes in this PR are a couple lines, but will immediately restore Natural Speech functionality.

changelist

  • hotfix: UTF-8 IO stream for piper, prevent crash, correctly speak ä accents
  • remove rich-text-modifier tags from ChatMessages
  • expand alphanumeric detection to accept diacritics alphabet

- UTF-8 IO stream for piper, prevent crash, correctly speak ä accents
- remove rich-text-modifier tags from ChatMessages
- expand alphanumeric detection to accept diacritics alphabet
@runelite-github-app
Copy link

runelite-github-app bot commented May 27, 2024

@Adam-
Copy link
Member

Adam- commented May 27, 2024

FWIW, RuneLite does not launch with -Dfile.encoding=windows-1252. It is possible that is the default encoding on the system and so the JVM uses it (that is even likely maybe). But it is certainly not an argument we are passing.

@phyce
Copy link
Contributor

phyce commented May 27, 2024

This might have been a windows update changing something. I had to run chcp 65001 in powershell in order to be able to use special characters like ñ without piper crashing.

@TheLouisHong
Copy link
Contributor Author

additional change

c7a5e0d63b18e8214acd35ed0fbd332eefc6c907

  • moved removeTags to only apply to notification type ChatMessages
    (we call them "system" internally)

@Nightfirecat Nightfirecat merged commit 8f3fe49 into runelite:master Jun 4, 2024
2 checks passed
frolv pushed a commit to blert-io/plugin-hub that referenced this pull request Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants