EverythingSmartHome / everything-presence-one

Official Repo for the Everything Presence One sensor!
383 stars 68 forks source link

$JYRPO declaration issue? #119

Open svente08 opened 8 months ago

svente08 commented 8 months ago

Hi all,

Trying to get 1.3.0b running. Installation works but multi target distance not really.

Could declaration be the problem?

logs_esphome-web-8bd630_run.txt

Thx in advance

EverythingSmartHome commented 8 months ago

Did you try toggling UART target off, waiting 10s then turning back on again?

svente08 commented 8 months ago

No success. As soon as I activate a second zone, only inconsistent values are displayed. Single target setup works.

EverythingSmartHome commented 8 months ago

Can you post logs and detail what the issue your seeing is?

svente08 commented 8 months ago

Attached logfiles and HA screenshot. (moving target more than 2 meters away)

Archive.zip

(no bh1750/shtcx sensor connected)

madninjaskillz commented 7 months ago

Simple work around for that; change if (line.substr(0, 6) == "$JYRPO") { to if (line.substr(1, 6) == "JYRPO") {

I'm not sure if the substr here is end or number of characters - if its the latter, you'll need to change the 6 to a 5 for the functionality to still work, but at the least, this change DOES get rid of the warning.

Djelibeybi commented 7 months ago

It needs to be if (line.substr(1, 5) == "JYRPO") which at least gets me some values, ~but I'm not entirely sure if they're correct~. The values appear to be correct if you configure the targets correctly. :)

svente08 commented 6 months ago

I got rid of the error message - thx for that. But the values still not correct. Target 1 ist always active and shows the end value of target.

@Djelibeybi What do you mean by configure correctly... Talking about start and end points of each beginning with target 1 and so on? Or Is there anything else to consider?