Closed PropGit closed 7 years ago
Unfortunately, most of those messages are printed by os_printf which, as far as I know, isn’t switchable. They should probably all be switched over to some new function like dbg_printf that maybe would take an extra argument indicating the type of message along with a way to select specific messages to print. That way you could, say, enable all CMD handling messages or all wi-fi status messages, or all HTTP messages, etc.
On Oct 13, 2016, at 5:09 PM, Parallax Git Administrator notifications@github.com wrote:
@dbetz https://github.com/dbetz - Can you think of a way that we could set DBG to be optional and disabled by default? Some way that doesn't prevent us from using DBG for our uses (development) in all the cases we need it to be?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/parallaxinc/Parallax-ESP/issues/9, or mute the thread https://github.com/notifications/unsubscribe-auth/AAq-1kq1lVxH6yPwsm4aGxXy-l2BR3I2ks5qzp4ggaJpZM4KWXzA.
Okay, let's consider this for Release 2, among others.
Update - Andy determined that he could set the DBG to input via a CHECK:gpio_xxx command at the start of the PBASIC example code... that allows us to use P12 in the BOE for servo control and it won't be affected by DBG. Perfect; problem solved.
So, the ability to filter DBG output is how this issue can proceed, but it's purely helpful for development purposes and no longer an issue for our tutorials or for users. It can proceed at a pace conducive to it's need for us developers.
This is done. There is a setting that enables/disables output to the DBG pin. The default is "disabled". There is also a "Debug Log" page in the web interface that shows the most recent 1400 bytes of debug output.
Closing.
UPDATED
@dbetz - Can you think of a way that we could set DBG to be optional and disabled by default? Some way that doesn't prevent us from using DBG for our uses (development) in all the cases we need it to be?We solved this immediate need with this solution.