Change my program to the new structure #1
Replies: 5 comments 22 replies
-
|
Hello again, @Hieromon I think I'm going to change my request for the reason that if it works correctly,
I see that to add the variables from one EdgeDriver to another EdgeDriver just add xxxxx.h ? /*
EgeDriver handling MQTT as an example of combining to EdgeUnified.
MyGpio.cpp is an implementation of GPIO EdgeDriver. EdgeDriver basically
*/
#include <Arduino.h>
#if defined(ARDUINO_ARCH_ESP8266)
#include <ESP8266HTTPClient.h>
#elif defined(ARDUINO_ARCH_ESP32)
#include <HTTPClient.h>
#endif
#include <ArduinoJson.h>
#include <AutoConnect.h>
#include <PubSubClient.h>
#include "EdgeUnified.h"
#include "MyMqtt.h"
#include "MyGpio.h" // <-----------------so far everything is working fine |
Beta Was this translation helpful? Give feedback.
-
Basically, yes. Please remember to |
Beta Was this translation helpful? Give feedback.
-
Hmmm, what was the response code for the POST request? |
Beta Was this translation helpful? Give feedback.
-
|
@FRANAIRBUS So, I have a few questions.
|
Beta Was this translation helpful? Give feedback.
-
this is from the autoconnect library this is from the autoconnect library. if in the end you are going to implement the saving of credentials in the file system remember to implement a way to delete them as you currently do (trash can symbol) in autoconnect menu "open SSIDs"
I believe that it is implicit in linear programming but perhaps the name should be changed. the Canarian expression is "it falls off mature" "it is omitted" "it is taken for granted" I don't know what to say it is something very basic if there is a complicated mathematical operation in the first loop it will not happen until it is completed causing an unexpected time maybe I should have put "0" in my report to avoid confusion. void startMQTT() {
Serial.println("Starting MQTT");
mqtt.data.SendMySQL = 0 * 60000L;
}In my report I wanted to express more the stability than the real time in milliseconds. In my case, the stability of the processes prevails more than precision over time. I will comment on a phrase that made me change my way of seeing the future completely "Imagine that a robot has the ability to be an engineer. That robot could imagine 1000 bridges and make the structural calculation in seconds, discarding the imperfect ones and finishing the entire project in a few minutes" So unless we have an ESP with a multiprocessor or a quantum ESP512 that would have more capacity than all the domestic computers in Europe combined, I think it is unlikely to achieve a flicker of exactly 1 millisecond. neither the human eye can see it nor my brain has the ability to multitask. if it is true that for the pairing in homekit I had to stop all the processes like send MySQL or increase the time to 10 minutes for example since the homekit webserver needs a time to pair without iterruptions I understand that there will be no major problem with EDGE, even so, some process interruption can be scheduled before updating by OTA I'll try this last one but I still haven't implemented the homekit library with EDGE, as soon as I have it I'll make a report. |
Beta Was this translation helpful? Give feedback.
-
Thank you. |
Beta Was this translation helpful? Give feedback.
-
|
not so much the word but less experienced people like me should understand that this can be skipped. // void setup
void startMQTT() {
Serial.println("Starting MQTT");
mqtt.data.SendMqtt = mqtt.data.timerMqtt.toFloat() * 60000L;
}
// void loop
void processMQTT() {
if (mqtt.data.serverNameMqtt.length()) {
if (millis() - mqtt.data.publishInterval > mqtt.data.SendMqtt) {
Serial.println("Starting processMQTT");
My_Mqtt_LOOP();
mqtt.data.publishInterval = millis();
}
}
}and you can use just this if you want. // void setup
void startMQTT() {
Serial.println("Starting MQTT");
mqtt.setInterval(mqtt.data.timerMqtt.toFloat() *60000L);
}
// void loop
void processMQTT() {
if (mqtt.data.serverNameMqtt.length()) {
My_Mqtt_LOOP();
}
}In my case I must change them to minutes because it would be difficult for my technicians to work in milliseconds ( X * 60000L ). My proposals for setInterval are: mqtt.setEdgeInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setProcessInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setDriverInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setEdgeDriverInterval(mqtt.data.timerMqtt.toFloat() *60000L); and the one I like the most is the first one, it is explicit and short
thanks again for your mention. |
Beta Was this translation helpful? Give feedback.
-
|
@FRANAIRBUS, Thanks for the suggestions. mqtt.setEdgeInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setProcessInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setDriverInterval(mqtt.data.timerMqtt.toFloat() *60000L);
mqtt.setEdgeDriverInterval(mqtt.data.timerMqtt.toFloat() *60000L);
Agree. I change it. |
Beta Was this translation helpful? Give feedback.
-
|
wow...i am very happy with your library and it has captured exactly what we needed. Every time I get deeper into programming with this ability to be able to use what I need or request many more things on demand, I like more this form of structure, although at first it costs a little more to adapt. I would like your opinion on this? // AutoConnectAux handler. Obtain AutoConnectElement values, copy to EdgeData
String auxMENUStart(AutoConnectAux& aux, PageArgument& args) {
const AutoConnectAux* requestAux = portal.aux(portal.where());
bool checked_Use_MySQL = ((AutoConnectAux&)(*requestAux))["Use_MySQL"].as<AutoConnectCheckbox>().checked;
menu.data.Use_MySQL = checked_Use_MySQL;
bool checked_Use_HOMEKIT = ((AutoConnectAux&)(*requestAux))["Use_HOMEKIT"].as<AutoConnectCheckbox>().checked;
menu.data.Use_HOMEKIT = checked_Use_HOMEKIT;
menu.data.hostName = aux["hostName"].as<AutoConnectText>().value;
menu.data.COMAND = aux["COMAND"].as<AutoConnectText>().value;
// You make EdgeData persistent when the value it has is changed.
menu.save();
menu.start();
return String();
}if (menu.data.Use_MySQL){
mysql.autoRestore(true);
mysql.serializer(mysqlSerialize, mysqlDeserialize, 512);
Edge.attach(mysql);
Edge.join(FPSTR(SETTINGS_MySQL), auxMySQLSetting);
Edge.join(FPSTR(START_MySQL), auxMySQLStart);
Edge.join(FPSTR(CLEAR_MySQL), auxMySQLClear);
Edge.join(FPSTR(STOP_MySQL), auxMySQLStop);
}although thanks to its API enable, I can think of infinite possibilities between EdgeDriver that with my previous structure would have been very complicated or impossible. void loop() {
mysql.enable(menu.data.Use_MySQL);
Edge.process(portal);
mDNSUpdate();
portal.handleClient();
}Everything is running smoothly as we go. please check this in your example in case it is necessary to define hostname for eps32, I see that in your original example you use hostname for the APID only and for the name I imagine in ThingSpeak but it doesn't change in MDNS. I don't know if he has it set like that for some reason? look: void onWiFiConnect(IPAddress& ip) {
String tcpName = "LH-" + menu.data.hostName;
const char* hostName = GET_HOSTNAME();
if (menu.data.hostName.length()) {
hostName = tcpName.c_str();
}
Serial.println("connected:" + WiFi.SSID());
Serial.println("IP:" + WiFi.localIP().toString());
Serial.printf("mDNS responder %s.local start", hostName);
if (MDNS.begin(hostName)) {
MDNS.addService("http", "tcp", 80);
Serial.println("ed");
}
else {
Serial.println(" failed");
}
}
|
Beta Was this translation helpful? Give feedback.
-
It is an expected problem, and we cannot say we have no way of addressing it. One solution is the approach I have presented on the #3 topic. The Arduino MySQL Connector makes a native connection to the MySQL server via a TCP connection. Thus, it does not require an HTTPClient instance to transparently access an HTTP session, contributing to a reduction in memory consumption. In this way, it may be possible to reduce the number of AutoConnectAux and HTTPClient instances resident in memory and implement complete processing even with the ESP8266's meager RAM capacity. The significance of #3 topic is to explore that possibility. If you really want to reduce memory consumption, you need to abolish INSERT by HTTP session transparency and change to a structure that narrows down the instances required for the transaction and allocates them dynamically. I would extend EdgeUnified to be able to dynamically instantiate EdgeDriver to match the changed structure. In other words, I can respond once you have successfully narrowed down your problem. First you need to narrow down the problem to a more specific state. |
Beta Was this translation helpful? Give feedback.
-
|
Hello @Hieromon so I'm sure I was wrong and using a query instead of using htpp.post will reduce memory consumption. It was something planned to change because I would like to read the database and compare, if setpoint has changed within the database then we change the setpoint variable within my sketch or turn on/off Rele. I am already changing my EdgeDriver mysql from post method to querry insert method anyway my chain is something big and it will stay like this char INSERT_DATA[] = "INSERT INTO %s.%s (Name, Location, Temp1, Temp2, Temp3, Hum1, Hum2, Hum3, Set1, Set2, Set3, Rele1, Rele2, Rele3, Rele4, Aux1, Aux2, In1, In2, In3, In4, AL0, AL1, AL2, AL3, AL4, AL5, AL6, AL7, AL8, IP)
VALUES ('%s',%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s,%s)";
ALARMA = AL1+AL2.....AL8;
AL0 = 00000000;
maybe I can reduce the alarms to a single variable with 8 digits and interpret it outside and adjust values so that they are not all a string but in the first instance there are 33 variables that I currently send by the htpp.post method I don't quite understand what you mean by dynamic instances |
Beta Was this translation helpful? Give feedback.
-
|
Test Summary I already have implemented the option to send data in both ways I have not noticed many changes in the use of memory, but I have noticed a delay in connection failure with a larger server in INSERT TO http.post respond error immediately
a total of 12 seconds delay on average |
Beta Was this translation helpful? Give feedback.
-
You need to adjust the |
Beta Was this translation helpful? Give feedback.
-
perfect solved
and
Thanks a lot 👍 |
Beta Was this translation helpful? Give feedback.
-
|
hello @Hieromon how are you How are you doing with this repository? any news or progress? |
Beta Was this translation helpful? Give feedback.
-
|
@FRANAIRBUS, I am currently prioritizing release preparation for AutoConnect 1.4.0. Once this work is complete, I expect to begin revising the program structure for EdgeUnified. This will allow us to deploy EdgeDriver on a second core. |
Beta Was this translation helpful? Give feedback.
-
|
Hi, @Hieromon how are you. I have seen you very busy answering autoconnect and I did not want to disturb |
Beta Was this translation helpful? Give feedback.
-
|
Hello @FRANAIRBUS, I remember it well. We have been on pause for a while to update Edge-Unified, but it may be a good time to get back into it. AutoConnect has become a bit too heavy, and with the update of the ESP32 Arduino core v.2.0.6, the core has increased in size and will no longer fit in the default partition. However, we still have homework to do. Continuation of this topic, right? |
Beta Was this translation helpful? Give feedback.
-
|
Yes, although as far as I can remember all the Edge-Unified tests have been a success The idea of a ping was that the final client could easily configure its ping address, whether it was a local server or Google itself. #include <AutoConnect.h> #include "LH_PING.h" |
Beta Was this translation helpful? Give feedback.
-
Hello @Hieromon
interesting library, seems to be very clean but I have to change many things including my mentality.
I have a problem here and I don't know why, it doesn't send me data to MySQL it returns error 1 I can't find what I'm missing
Beta Was this translation helpful? Give feedback.
All reactions