Skip to content

Default to celsius temperature unit #131

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ypid
Copy link
Contributor

@ypid ypid commented Apr 19, 2025

@ypid ypid force-pushed the feature/default-to-celsius branch from f9367e9 to 7a8db3c Compare April 29, 2025 12:50
@MallocArray
Copy link
Owner

I'm still pondering this. I'm in the US and for better or worse, the default unit here is F. It should be a one-time change and not a big deal, but I believe over half of the AirGradient units are located in the US, although I can't speak to which ones are running this ESPHome based firmware vs the stock firmware.

@ypid
Copy link
Contributor Author

ypid commented Apr 29, 2025

I feel that in scientific communities, SI units are dominant. I guess AirGradient also defaults to Celsius:

Would be interesting what https://app.airgradient.com/ defaults to. I did not try because I did not find a "delete user account" button to clean up the test user again.

https://en.wikipedia.org/wiki/Fahrenheit gives a nice overview which countries still default to Fahrenheit ;-)

@MallocArray
Copy link
Owner

I believe that the Dashboard changes the default depending on what country you are configured with.

I know the AirGradient repo's ESPHome config defaults to C, as that was the one thing they changed when they forked from this repo.

I know it probably should default to C, just give my American stubbornness a bit to soften. ;)

@lightingman117
Copy link

lightingman117 commented Apr 29, 2025 via email

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