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

fix: update get vocalization in the isolate #1225

Merged
merged 17 commits into from
Mar 27, 2024
Merged

Conversation

jessielw
Copy link
Collaborator

@jessielw jessielw commented Mar 21, 2024

Fixes #1188

lib/main.dart Outdated Show resolved Hide resolved
feat: create ttsmodel directly inside of the isolate and update it in real time based on the prefs listener
feat: now updates isbotmuted, isemotemuted, ispreludemuted, israndomvoiceenaabled, pitch, and speed
feat: uses a new method in TextToSpeechPlugin that accepts pitch & speed to update the native code
refactor: getVocalization accepts isprelude muted directly from the ttsmodel instead of a bool variable
…e with pitch/speed

refactor: debug print all potential errors in TextToSpeechPlugin
…uted, _isEmoteMuted, _isPreludeMuted, _speed, and _pitch

refactor: removed code to create an empty constructor
@jessielw jessielw requested a review from kevmo314 March 26, 2024 19:46
lib/tts_isolate.dart Outdated Show resolved Hide resolved
refactor: add an internal class method by updateFromJson and TtsModel.fromJson
refactor: removed setters setPitch, setSpeed, setPreludeMuted, setEmoteMuted, setBotMuted, and setRandomVoiceEnabled
Copy link
Contributor

@kevmo314 kevmo314 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

lib/tts_isolate.dart Outdated Show resolved Hide resolved
lib/tts_isolate.dart Outdated Show resolved Hide resolved
@jessielw jessielw requested a review from kevmo314 March 27, 2024 00:05
Copy link
Collaborator

@SputNikPlop SputNikPlop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for taking care of the android bit

@SputNikPlop SputNikPlop changed the title Update get vocalization in the isolate fix: update get vocalization in the isolate Mar 27, 2024
lib/tts_isolate.dart Outdated Show resolved Hide resolved
@jessielw jessielw merged commit c32a12e into main Mar 27, 2024
4 checks passed
@jessielw jessielw deleted the update-get-vocalization branch March 27, 2024 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update getVocalization() to use this function
3 participants