search

3KUdelta / Solar_WiFi_Weather_Station

Based on the work of Open Green Energy: First solar Zambretti Forecaster
111 stars 49 forks source link

Fahrenheit ? #4

Closed bhamilton1co closed 5 years ago

bhamilton1co commented 5 years ago

Hello, any chance you can add an option to settings.h to change from Celsius to Fahrenheit? Would love to try your project out, but need the readings in Fahrenheit.

bhamilton1co commented 5 years ago

​I need Fahrenheit for my Temperature readings.​ Do you see any issues with changing this line of code:

From: measured_temp = measured_temp + TEMP_CORR;

And Changing To: measured_temp = measured_temp *1.8+32;

It seems to be working. Let me know if you think this will cause any issues, or if you know of a better way to implement Fahrenheit Temperature readings.

Thanks.

3KUdelta commented 5 years ago

Hi Brian - sorry for the late answer. Your proposal will work without any problems. Just in case you want to correct the reading with the TEMP_CORR value then simply use both lines. e.g. measured_temp = measured_temp + TEMP_CORR; // correcting the celsius value measured_temp = measured_temp *1.8+32; // converting it into Fahrenheit Marc

bhamilton1co commented 5 years ago

Perfect, Thanks!

From: 3KU_Delta notifications@github.com Sent: Monday, July 29, 2019 2:02 PM To: 3KUdelta/Solar_WiFi_Weather_Station Solar_WiFi_Weather_Station@noreply.github.com Cc: Brian Hamilton brian@liminis.com; Author author@noreply.github.com Subject: Re: [3KUdelta/Solar_WiFi_Weather_Station] Fahrenheit ? (#4)

Hi Brian - sorry for the late answer. Your proposal will work without any problems. Just in case you want to correct the reading with the TEMP_CORR value then simply use both lines. e.g. measured_temp = measured_temp + TEMP_CORR; // correcting the celsius value measured_temp = measured_temp *1.8+32; // converting it into Fahrenheit Marc

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/3KUdelta/Solar_WiFi_Weather_Station/issues/4?email_source=notifications&email_token=AD5HRSZROVUR336FMHOOQHDQB5EB3A5CNFSM4IHJVPLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3B24JY#issuecomment-516140583, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD5HRS6AQZR7IGE54XHQ7PLQB5EB3ANCNFSM4IHJVPLA.