nlrb / com.weather-sensors

Wireless weather sensor app voor Athom Homey
https://apps.athom.com/app/com.weather-sensors
MIT License
14 stars 24 forks source link

Wrong temperature #13

Closed KRWaara closed 6 years ago

KRWaara commented 6 years ago

I have a temperature sensor from Telldus: telldus_temperatursensor_kylfrys_med_display_tellstick-_och_rfxtrx433e-kompatibel_3 When I add the sensor to HOMEY it finds a sensor called WS-1050. It uses a protocol Alecto V1. See settings here: settings.

The thermistor is in -24 degrees C, but it reports an temperature way off as depicted below. The battery icon alarm is blinking. And yes, the batteries are brand new.

Bug?

nlrb commented 6 years ago

Problem is that there are multiple sensors (e.g. Telldus & Alecto) that use the same signals, but with different definitions. There is no way to tell them apart from the signal itself. Your sensor doesn't have a battery indication, the Alecto one does.

Can you set the offset in the device setting to get the correct temperature?

KRWaara commented 6 years ago

So What you are telling me is that this sensor isn't going to work because the app cannot tell wether it is a TELLDUS or an ALECTO. Ever?

Yes, I can adjust the offset to get a near exact match. But as soon as the actual temperature changes the displayed temperature goes totally off. Cannot find out what's really happening.

About the battery indicator blinking red: Strange that the app reads a "Batttery low" - message where here isn't any.

This is only one problem among others with HOMEY.. I think I made a bad purchase.. :(

R

2017-11-03 16:23 GMT+01:00 Ramón Baas notifications@github.com:

Problem is that there are multiple sensors (e.g. Telldus & Alecto) that use the same signals, but with different definitions. There is no way to tell them apart from the signal itself. Your sensor doesn't have a battery indication, the Alecto one does.

Can you set the offset in the device setting to get the correct temperature?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nlrb/com.weather-sensors/issues/13#issuecomment-341735134, or mute the thread https://github.com/notifications/unsubscribe-auth/AfyiHDoPnBbT0tKiiNQyw8mMz75m7Yfiks5syy_VgaJpZM4QQdtx .

--

========================= Roger Waara

nlrb commented 6 years ago

BTW: which version of the app are you using? The one from the app store or the latest code on GitHub? Try the latest code on GitHub, That has a different algorithm to decide what type of sensor it is. That might work better for you.

KRWaara commented 6 years ago

I'm sorry to say that I use the version from Github. some one week old. I can add other sensors (temp/humidity) from TELLDUS absolutely flawless. It's this temeperature sensor that's killing me and, as I wrote, ultimately is going to force me to leave HOMEY. Too much problems, and I haven't even tried to add my switch from NEXA into the system yet... :-/.

R

2017-11-04 20:21 GMT+01:00 Ramón Baas notifications@github.com:

BTW: which version of the app are you using? The one from the app store or the latest code on GitHub? Try the latest code on GitHub, That has a different algorithm to decide what type of sensor it is. That might work better for you.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nlrb/com.weather-sensors/issues/13#issuecomment-341923089, or mute the thread https://github.com/notifications/unsubscribe-auth/AfyiHMrU1pMZbe1oS10nhm4f-QmP51Biks5szLlBgaJpZM4QQdtx .

--

========================= Roger Waara

nlrb commented 6 years ago

If you want to leave Homey that is fine with me. I don't work for Athom, so I don't care. This kind of attitude does not motivate me to help you.

As workaround you can change line 127 of node_modules/alecto/index.js to FineOffset = true.

KRWaara commented 6 years ago

Thank you for the tip. Sorry for my "attitude". It's just that I'm so tired..

R

Den 5 nov. 2017 16:44 skrev "Ramón Baas" notifications@github.com:

If you want to leave Homey that is fine with me. I don't work for Athom, so I don't care. This kind of attitude does not motivate me to help you.

As workaround you can change line 127 of node_modules/alecto/index.js to FineOffset = true.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nlrb/com.weather-sensors/issues/13#issuecomment-341982517, or mute the thread https://github.com/notifications/unsubscribe-auth/AfyiHBY6Nu6mdMQCboYdRQhpXJe6JX7Tks5szdfPgaJpZM4QQdtx .

KRWaara commented 6 years ago

At home now. See previous mail from you..

Where do I find this line node_modules/alecto/index.js to FineOffset = true.

Roger

2017-11-05 20:24 GMT+01:00 Roger Waara roger.waara@gmail.com:

Thank you for the tip. Sorry for my "attitude". It's just that I'm so tired..

R

Den 5 nov. 2017 16:44 skrev "Ramón Baas" notifications@github.com:

If you want to leave Homey that is fine with me. I don't work for Athom, so I don't care. This kind of attitude does not motivate me to help you.

As workaround you can change line 127 of node_modules/alecto/index.js to FineOffset = true.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nlrb/com.weather-sensors/issues/13#issuecomment-341982517, or mute the thread https://github.com/notifications/unsubscribe-auth/AfyiHBY6Nu6mdMQCboYdRQhpXJe6JX7Tks5szdfPgaJpZM4QQdtx .

--

========================= Roger Waara