schmurtzm / MrDiy-Audio-Notifier

MrDiy Audio Notifier (based on esp8266audio) ported to platformio for ESP32
58 stars 23 forks source link

Version 0.6 doesn't support multiple notifiers #9

Closed Hurstolds closed 2 years ago

Hurstolds commented 2 years ago

After some further testing, here is what I’m seeing. I am running two of these speaker notifiers. Speaker 1 was running firmware 0.4 Speaker 2 was running firmware 0.5

I upgraded speaker 2 to version 0.6 and it works, just samvoice is broken.

I just now upgraded speaker 1 to version 0.6 and there’s a problem running both at version 0.6 This was an issue I saw when running Mr DIY original firmware one more than one speaker. It’s basically created a MQTT storm, where both units keep firing off messages saying they’re idle and connected. When this happens, I can’t send commands to either speaker.

To further test, I downgraded speaker 1 to version 0.5. speaker 2 running version 0.6 and it has the same problem. Both are firing messages non stop via MQTT, causing neither of them to work.

So I stepped speaker 1 back down to version 0.4 . Speaker 2 is running latest 0.6 version, and now both work as expected. I presume version 0.5 introduced an issue that causes multiple notifiers to break.

One further test to confirm:

If both speaker/notifiers are running firmware version 0.4, they both work correctly. No issues with contention and no MQTT storm of broadcasts from the notifiers.

schmurtzm commented 2 years ago

Yes I see what happens here. You can solve it by changing the "AP SSID" name with different name for your 2 devices..

Before the name was different for each device, for simplicity I put an unique name (it allow to access to your device with "http://mrdiynotifier.local/") but it was a bad idea.

This "AP SSID" field in realty is used for 3 different things :

I will revert back this modification on next version.

schmurtzm commented 2 years ago

0.61 released, it should support multiple notifiers again. May be you will need to erase flash to generate new names.

Hurstolds commented 2 years ago

All seems well, after testing 0.61! Thank you, I did not have to do a full flash. I was able to flash via GUI updater and it kept all my previous settings. Great job and thank you again!