Comments (3)
@presslab-us were you seeing this on your end while testing? I'm seeing random characters in what is being sent. I'm guessing one of the arrays is getting filled with null and need some kind of termination instead.
I reverted to the previous code (with Strings) in mqtt_publish, and it worked without issue.
from expandable-6-channel-esp32-energy-meter.
I checked out master and tested it and I didn't see any problems. What text are you using for base topic and prefix? I am using "emon/main" for base topic, and I tested both "" and "123" for prefix.
In any case, I did see a potential bug. Because sprintf()
has variable arguments the compiler may not know to properly cast the String
to a char *
. Why don't you try this and see if it helps:
diff --git a/Software/EmonESP/src_6chan/mqtt.cpp b/Software/EmonESP/src_6chan/mqtt.cpp
index c80b2fd..c5a9f57 100644
--- a/Software/EmonESP/src_6chan/mqtt.cpp
+++ b/Software/EmonESP/src_6chan/mqtt.cpp
@@ -101,7 +101,7 @@ void mqtt_publish(const char * data)
{
const char * data_ptr = data;
char * topic_ptr = mqtt_topic_prefix;
- topic_ptr += sprintf(mqtt_topic_prefix, "%s/%s", mqtt_topic, mqtt_feed_prefix);
+ topic_ptr += sprintf(mqtt_topic_prefix, "%s/%s", mqtt_topic.c_str(), mqtt_feed_prefix.c_str());
do
{
from expandable-6-channel-esp32-energy-meter.
this was fixed in the latest version
from expandable-6-channel-esp32-energy-meter.
Related Issues (20)
- Help setup 6Channel Board HOT 18
- Feature Request: Add throttle_average
- Platformio Build for emonesp-nodemcu32s fails; two reasons HOT 2
- ATM90E32 not communicating ESP32C3 HOT 4
- 6Ch crash on brownout HOT 1
- Can I make a ESP8266 work with jumpers? HOT 1
- 3 Phase typo HOT 2
- 3 phase transfo board HOT 2
- AddOn Boards Power Calculation Wildly Incorrect HOT 8
- Bigger board HOT 2
- Updated files available?
- Second Voltage and Frequency Reading Zero?
- Update YAMLs to conform to Home Assistant developer guidelines? HOT 3
- CT Selection Query: Suggestions for Measuring 600A Current with ATM90E36 IC (Based on Datasheet) HOT 1
- With US split phase, simultaneous import and export is possible - small improvement to example HOT 1
- Two channels reporting erratically at certain times/conditions HOT 1
- Replacement for ESP32-DevKitC-32U, which is NRND? HOT 2
- Current Scaling Not Correct HOT 5
- esp32 wroom 32d loosing configuration details
- I see this log message after updating esphome firmware today. HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from expandable-6-channel-esp32-energy-meter.