12-11-2019, 10:02 PM
I will test this later this week.
|
Thread Rating:
[Solved] Strange receive problem with 433Mhz Receiver and pilight 8.1.5
|
What do the logs say when it staggers after applying this patch:
This patch could also be usefull instead of the previous one:
This should be even more informative at the moment. It tells us when the wiringx timer is actually delayed or not as you suggest.
And can you check how pilight-raw performs when started like this:
Code:
diff --git a/daemon.c b/daemon.c
index 7a085c92..29bf93cc 100644
--- a/daemon.c
+++ b/daemon.c
@@ -2631,11 +2631,13 @@ void openconsole(void) {
#endif
static void pilight_abort(uv_timer_t *timer_req) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
exit(EXIT_FAILURE);
}
#ifdef MQTT
static void ping(uv_timer_t *handle) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
mqtt_ping(handle->data);
}
@@ -2754,6 +2759,7 @@ static void mqtt_callback(struct mqtt_client_t *client, struct mqtt_pkt_t *pkt,
#endif
static void pilight_stats(uv_timer_t *timer_req) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
int watchdog = 1, stats = 1;
// double itmp = 0.0;
{
@@ -2822,6 +2828,7 @@ static void pilight_stats(uv_timer_t *timer_req) {
}
static void generate_uuid(uv_timer_t *timer_req) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
int nrdevs = 0, x = 0;
char **devs = NULL, *p = NULL;
if((nrdevs = inetdevs(&devs)) > 0) {
diff --git a/libs/pilight/core/http.c b/libs/pilight/core/http.c
index a57899cb..19440db1 100644
--- a/libs/pilight/core/http.c
+++ b/libs/pilight/core/http.c
@@ -542,6 +542,8 @@ static void poll_close_cb(uv_poll_t *req) {
}
static void timeout(uv_timer_t *req) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
struct request_t *request = req->data;
void (*callback)(int, char *, int, char *, void *) = request->callback;
void *userdata = request->userdata;
diff --git a/libs/pilight/core/mqtt.c b/libs/pilight/core/mqtt.c
index 818c88e4..27fd1df4 100644
--- a/libs/pilight/core/mqtt.c
+++ b/libs/pilight/core/mqtt.c
@@ -1014,6 +1014,8 @@ int mqtt_pkt_disconnect(struct mqtt_pkt_t *pkt) {
}
static void mqtt_client_timeout(uv_timer_t *handle) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
#ifdef _WIN32
uv_mutex_lock(&mqtt_lock);
#else
@@ -1035,6 +1039,8 @@ static void mqtt_client_timeout(uv_timer_t *handle) {
}
static void mqtt_message_resend(uv_timer_t *handle) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
#ifdef _WIN32
uv_mutex_lock(&mqtt_lock);
#else
diff --git a/libs/pilight/core/ntp.c b/libs/pilight/core/ntp.c
index 3122d564..361d34fa 100644
--- a/libs/pilight/core/ntp.c
+++ b/libs/pilight/core/ntp.c
@@ -100,6 +100,8 @@ static void close_cb(uv_handle_t *handle) {
}
static void ntp_timeout(uv_timer_t *param) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
/*
* Make sure we execute in the main thread
*/
@@ -126,6 +128,8 @@ static void ntp_timeout(uv_timer_t *param) {
}
static void restart(uv_timer_t *req) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
nr = 0;
ntpsync();
}
@@ -195,6 +199,8 @@ static void on_send(uv_udp_send_t *req, int status) {
}
static void loop(uv_timer_t *req) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
/*
* Make sure we are called from the main thread
*/
diff --git a/libs/pilight/lua_c/async/timer.c b/libs/pilight/lua_c/async/timer.c
index 0b2350f3..df59b26a 100644
--- a/libs/pilight/lua_c/async/timer.c
+++ b/libs/pilight/lua_c/async/timer.c
@@ -370,6 +370,7 @@ static void plua_async_timer_object(lua_State *L, struct lua_timer_t *timer) {
}
static void timer_callback(uv_timer_t *req) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
struct lua_timer_t *timer = req->data;
char name[512], *p = name;
memset(name, '\0', 512);
diff --git a/libs/pilight/lua_c/wiringx.c b/libs/pilight/lua_c/wiringx.c
index cc0fee08..ca5ed7fa 100644
--- a/libs/pilight/lua_c/wiringx.c
+++ b/libs/pilight/lua_c/wiringx.c
@@ -414,10 +414,13 @@ int plua_wiringx_pin_mode(struct lua_State *L) {
}
static void plua_wiringx_poll_timer(uv_timer_t *req) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
struct lua_wiringx_gpio_t *data = req->data;
int nr = data->data.rptr, idx = data->data.idx;
if(nr == 1) {
return;
}
diff --git a/libs/pilight/protocols/API/cpu_temp.c b/libs/pilight/protocols/API/cpu_temp.c
index 412aeaf0..6b0d9310 100644
--- a/libs/pilight/protocols/API/cpu_temp.c
+++ b/libs/pilight/protocols/API/cpu_temp.c
@@ -55,6 +55,8 @@ static pthread_mutex_t lock;
static pthread_mutexattr_t attr;
static void *thread(void *param) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
struct protocol_threads_t *node = (struct protocol_threads_t *)param;
struct JsonNode *json = (struct JsonNode *)node->param;
struct JsonNode *jid = NULL;
diff --git a/libs/pilight/protocols/API/datetime.c b/libs/pilight/protocols/API/datetime.c
index 6029c332..c1cee685 100644
--- a/libs/pilight/protocols/API/datetime.c
+++ b/libs/pilight/protocols/API/datetime.c
@@ -68,6 +68,8 @@ static char *format = NULL;
static pthread_mutex_t lock;
static void *thread(void *param) {
+
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
char UTC[] = "UTC";
struct protocol_threads_t *thread = (struct protocol_threads_t *)param;
struct JsonNode *json = (struct JsonNode *)thread->param;
diff --git a/libs/pilight/protocols/API/openweathermap.c b/libs/pilight/protocols/API/openweathermap.c
index 5603617a..4cd142ab 100644
--- a/libs/pilight/protocols/API/openweathermap.c
+++ b/libs/pilight/protocols/API/openweathermap.c
@@ -533,6 +533,7 @@ static void thread(uv_work_t *req) {
}
static void *update(void *param) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
/*
* Make sure we execute in the main thread
*/
diff --git a/libs/pilight/protocols/API/sunriseset.c b/libs/pilight/protocols/API/sunriseset.c
index 5ee75577..16bc48af 100644
--- a/libs/pilight/protocols/API/sunriseset.c
+++ b/libs/pilight/protocols/API/sunriseset.c
@@ -310,6 +310,7 @@ static unsigned long min(unsigned long a, unsigned long b, unsigned long c) {
}
static void *thread(void *param) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
uv_timer_t *task = param;
struct data_t *settings = task->data;
struct tm tm;
diff --git a/libs/pilight/protocols/API/wunderground.c b/libs/pilight/protocols/API/wunderground.c
index a39e3d19..ed3d74fe 100644
--- a/libs/pilight/protocols/API/wunderground.c
+++ b/libs/pilight/protocols/API/wunderground.c
@@ -659,6 +659,7 @@ static void thread(uv_work_t *req) {
}
static void *update(void *param) {
+ logprintf(LOG_DEBUG, "==== %s %d ====", __FILE__, __LINE__);
uv_timer_t *timer_req = param;
struct data_t *settings = timer_req->data;
This patch could also be usefull instead of the previous one:
Code:
diff --git a/libs/pilight/lua_c/wiringx.c b/libs/pilight/lua_c/wiringx.c
index cc0fee08..95c0c2f4 100644
--- a/libs/pilight/lua_c/wiringx.c
+++ b/libs/pilight/lua_c/wiringx.c
@@ -32,6 +32,10 @@
#include "../core/log.h"
struct lua_wiringx_t;
+struct {
+ unsigned long first;
+ unsigned long second;
+} timestamp;
typedef struct lua_wiringx_gpio_t {
struct lua_wiringx_t *parent;
@@ -417,10 +421,23 @@ static void plua_wiringx_poll_timer(uv_timer_t *req) {
struct lua_wiringx_gpio_t *data = req->data;
int nr = data->data.rptr, idx = data->data.idx;
+ struct timeval tv;
+ gettimeofday(&tv, NULL);
+
+ timestamp.first = timestamp.second;
+ timestamp.second = 1000000 * (unsigned int)tv.tv_sec + (unsigned int)tv.tv_usec;
+
+ int duration = (int)((int)timestamp.second-(int)timestamp.first);
+ if(duration < 248000) {
+ logprintf(LOG_NOTICE, "plua wiringx timer too delayed: %d", duration);
+ }
+
if(nr == 1) {
return;
}
+ logprintf(LOG_DEBUG, "number pulses gathered %d", nr);
+
data->data.idx ^= 1;
data->data.rptr = 1;And can you check how pilight-raw performs when started like this:
Code:
./pilight-raw --linefeed -C /etc/pilight/config2.1.json --pulsesperline 10
12-15-2019, 09:37 AM
thank you curlymo, i try today or tomorrow at latest
12-17-2019, 08:18 PM
I made some videos from my terminal with pilight-raw 8.1.4 and 8.1.5 nightly. You can see the difference. I'm very busy at the moment, so I can test your patch only in the next few days.
The files are too big as attachment, i uploaded them to a cloud:
https://frankyscloud.spdns.de/index.php/...fzjPLfTRa0
PW: pilight
The files are too big as attachment, i uploaded them to a cloud:
https://frankyscloud.spdns.de/index.php/...fzjPLfTRa0
PW: pilight
12-17-2019, 09:18 PM
I see, but i don't care about false positives like all the noise. I care about the false positives. If i look the second nr 7 of pilight815_nightly_pulseperline10.mp4 i see what i want to see, the reception of a possible true signal.
12-18-2019, 11:09 AM
yes, with pilight-receive 8.1.5 there are a lot of false positives namely "secudo-smoke-sensor". With 8.1.4 not so often or very rare.
Browsing: 1 Guest(s)