Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quite Stable #66

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Quite Stable #66

wants to merge 9 commits into from

Conversation

ammuller
Copy link

Works very well :)

Only issue is that sometimes after a wifi reconnect the webpage works but the lights doesn't.

Michael Muller and others added 8 commits September 22, 2017 11:40
 issue 62 and 50
 possibly adresses settings corrupted by not formating the filesystem if it is
 formatted
 This possibly could lead to corruption

Fix Settings Corruption
Fixes mtongnz#50
Fixes Setting not saving / corruption
Should address corruption issues
Format of eprom logic was incorrect
 issue 62 and 50
 possibly adresses settings corrupted by not formating the filesystem if it is
 formatted
 This possibly could lead to corruption

Fix Settings Corruption
Fixes mtongnz#50
Fixes Setting not saving / corruption
Should address corruption issues
Format of eprom logic was incorrect

Preforming quite stable, after wifi reconnect the webpage always works,
but the lights not always
Preforming quite stable, sometimes lights stop working after wifi
reconnect
Preforming quite stable, sometimes lights stop working after wifi
reconnect
@ammuller ammuller closed this Sep 26, 2017
@ammuller ammuller reopened this Sep 26, 2017
@mtongnz
Copy link
Owner

mtongnz commented Oct 3, 2017

@ammuller I am really looking forward to giving this a try when I get some time - hopefully this weekend. Thanks heaps for the work you've put in!

@ammuller
Copy link
Author

@mtongnz Hi, do you any feedback for me?

@donnib
Copy link

donnib commented Nov 27, 2017

Any news on this ?

@ammuller
Copy link
Author

ammuller commented Dec 4, 2017

@mtongnz Could you assist me? I would like to disable all the Dmx code to determine if the lockup is in the Dmx or WS code.

@bombcheck
Copy link

Still crashes very often when used with many pixel. With 99 it runs often many hours but with 484 it crashes after minutes. Any news on that?

@bombcheck
Copy link

bombcheck commented Mar 8, 2018

@ammuller

  • Hm checked your code: The wifi-connection-check in the main loop is useless I think, as we set WiFi.setAutoReconnect(true) in wifiStart(). It just eats up cpu cycles, disturbing the timing.
  • Checking the reset reason during setup is nice, but this way it only start doing dmx output if a normal start/reset occurs, not when the wdt fires or an exception occurs. So your system "freezes", not doing any visible output any more before a manual triggered reboot. The best working way is to do an ESP.restart() on those other cases. This way, the system does a clean restart if the watchdog fires or an exception occurs. Works quite well: System freezes for some seconds, then gets back to work.

@bombcheck
Copy link

@ammuller
And it seemed you killed the stand alone mode... :(

@ammuller
Copy link
Author

@bombcheck The reason for not using WiFi.setAutoReconnect(true) is it writes to the flash every time, not sure if relevant, but will wear out flash eventually.

@ammuller
Copy link
Author

@bombcheck if the wdt triggers, my thought at that moment was, since it happens regularly, there is something wrong (like running out of stack) when the code runs stable for extended times this can be revised. Please note: ESP.reset() is a hard reset and can leave some of the registers in the old state which can lead to problems, its more or less like the reset button on the PC.
as per
https://www.pieterverhees.nl/sparklesagarbage/esp8266/130-difference-between-esp-reset-and-esp-restart

@ammuller
Copy link
Author

Please hard Reset after flashing
see:
esp8266/Arduino#1017

@bombcheck
Copy link

I always do a hard reset after flashing. And I use ESP.restart(), knowing that ESP.reset() is not the first choice. :)
Did not found anything during a quick search about WiFi.setAutoReconnect(true) and excessive flash writes...

@mtongnz
Copy link
Owner

mtongnz commented Mar 25, 2018

My reasoning behind the setup checks for reset reason was more for debug when I was trying different settings - rather than having to do a complete reflash when it locked up, I'd be able to simply change the offending setting back via the GUI and then continue my fault finding. I intend to change this behavior to a reset as suggested when the bugs are worked out.
I've found in my testing that this code still seems to freeze a bit on me.
I'm hoping to do a bit of testing with the new pull here from @scamiv. It looks very promising and seems that he's found quite a few of the root causes. Give it a whirl and see how it goes - with more of us on this, hopefully we'll get it to a stable place sooner

@Sine-Audio
Copy link

Sine-Audio commented May 12, 2018

@scamiv I can compile and flash your version with the latest Arduino 1.8.5 and SDK 2.4.1, but it does not run.
The original version from Matt compiles, flashes and runs normally with the same board settings on the same board. I'm using a NodeMCU V2 (but I also tried Wemos D1 R2). I've cleaned up everything with a clean arduino install and only the libraries from Matt are present (and also Arduino JSON latest version).
I set the FF to 40mhz, CPU to 160mhz, lwIP variant to v2 Lower Memory. I've tried almost every baudrate setting too.
Does your version require specific library versions or anything else present on the ESP?

@tohox
Copy link

tohox commented May 13, 2018

@Sine-Audio please see #92 as well as #65

@Sine-Audio
Copy link

Sine-Audio commented May 13, 2018

@tohox I dont think its the same issue, in my case it just sets up the default hotspot with the generic ESP name without encryption and thats all. No response. I've tried like 6 boards and all the same issue, I've tried changing pins etc but it just does not run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants