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.

Sign up for GitHub

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

Jump to bottom

Weather: Option for more detail in text-to-speech #220

Open
yaomtc opened this issue Jul 17, 2024 · 2 comments · May be fixed by #271
Open

Weather: Option for more detail in text-to-speech #220

yaomtc opened this issue Jul 17, 2024 · 2 comments · May be fixed by #271
Labels
enhancement New feature or request good first issue Good for newcomers skill Skill request or bug

Comments

@yaomtc
Copy link

yaomtc commented Jul 17, 2024

Currently it only says "broken clouds" for example, which is helpful, but (optionally?) I'd like the temperature too. Yeah, you can just read the screen, but a "verbosity" setting could be helpful especially for the vision impaired.
@Stypox Stypox added enhancement New feature or request good first issue Good for newcomers skill Skill request or bug labels Aug 21, 2024
@Inhishonor
Copy link
Contributor

@Stypox Hey, can I try to tackle this? I think I have a way to do it. Should I try to implement a setting for it, or just add the temperature?

@primesun
Copy link

primesun commented Jan 4, 2025

Personally I'd vote for just adding it. The current temperature is one of the most important pieces of information and I expect the vast majority of people would want to hear it. If there's a need to make it configurable later, it can be a separate issue.

@Inhishonor Inhishonor linked a pull request Jan 10, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers skill Skill request or bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

This changes the weather output to provide more detail. Inhishonor/dicio-android
4 participants
@primesun @yaomtc @Stypox @Inhishonor