Skip to content

Conversation

@serprex
Copy link
Contributor

@serprex serprex commented Jun 6, 2025

  1. reading out every second doesn't have enough time to even say more than a number
  2. tts is hard to hear the rest of the game over while it's counting non stop

Counting up only occurs during Dampe Race & Horse Race

Build Artifacts

@serprex serprex changed the title tts: only announce timer at 30s tts: only announce timer at 30s intervals while counting up Jun 6, 2025
@serprex serprex force-pushed the less-seconds-tts branch from 640d1b0 to 6864f0a Compare June 6, 2025 05:04
1. reading out every second doesn't have enough time to even say more than a number
2. tts is hard to hear the rest of the game over while it's counting non stop
@serprex serprex force-pushed the less-seconds-tts branch from 6864f0a to 5a0d9e6 Compare June 7, 2025 00:53
@Pepper0ni
Copy link
Contributor

Given that the go up timers are 50 seconds and a minute respectively, I wonder if 30 seconds is the best interval because it's useless for epona race and only tells you you've failed in dampe. I'm not sure what number would actually be best though, maybe 25 (failed epona race and nearly out of time for dampe?)

IIRC running man race is also a go-up timer though not currently relevant for anything sounds shufflable to me though

@serprex
Copy link
Contributor Author

serprex commented Jun 8, 2025

I think 10s frequency when timer <= 60 would be fine

@serprex serprex changed the title tts: only announce timer at 30s intervals while counting up tts: only announce timer at 10s intervals Jun 8, 2025
@Malkierian Malkierian merged commit 9432b34 into HarbourMasters:develop Jun 12, 2025
6 checks passed
krazyjakee pushed a commit to krazyjakee/OOT that referenced this pull request Sep 6, 2025
* tts: only announce timer at 30s

1. reading out every second doesn't have enough time to even say more than a number
2. tts is hard to hear the rest of the game over while it's counting non stop

* under a minute announce every 10s
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.

3 participants