Qrome / marquee-scroller

Marquee Scroller Clock News Weather and More
https://www.thingiverse.com/thing:2867294
MIT License
324 stars 159 forks source link

Add temperature and errors checks #252

Open yordanov-alexandar opened 1 year ago

yordanov-alexandar commented 1 year ago

Hi I am making this project as a gift for my father. He is not a technical person so I added a check to not show the scrolling errors as they will only confuse him. In the past he has also expressed desire to see only the time and no weather info so I added another check to not show the temperature(the only weather info that is not already behind a checkbox option) scrolling. During the testing of these two additions I had a lot of crashes and subsequent restarts of the clock. After some googling and looking at the stacktrace (I have zero experience with C++ and Arduino) I decided that the problem is with string concatenations so I commented some of the Serial.println code and had no problems since then. This might (or might not :) ) solve some of the issues I saw here where people are having crashes. This for example sounds exactly like what was happening to me. I have no idea if that problem can be solved in another way. Here is the patch for these changes errors_temp_checks.zip