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

[Bug] Can't enable Firmware Debug log #1056

Closed
jsimmonds2 opened this issue Nov 8, 2019 · 8 comments · Fixed by #1070
Closed

[Bug] Can't enable Firmware Debug log #1056

jsimmonds2 opened this issue Nov 8, 2019 · 8 comments · Fixed by #1070

Comments

@jsimmonds2
Copy link
Contributor

Testing on Pre-Release v8.3.0-rc3

Expected fbos_config.firmware_debug_log to change as the FE Log Setting item was toggled.

Picture this :)

image

@gabrielburnworth
Copy link
Contributor

Try fbos_config.arduino_debug_messages.

@jsimmonds2
Copy link
Contributor Author

Oh, Ok and thanks.
Sorry for the noise.

Some refactoring going on that I hadn't bothered to follow through :(

@jsimmonds2
Copy link
Contributor Author

A deeper look tells me that bug(s) lurk.

E.g https://github.com/FarmBot/farmbot_os/blob/staging/farmbot_core/lib/farmbot_core/firmware_side_effects.ex

  should_log? = Asset.fbos_config().firmware_debug_log

:-)

@ConnorRigby
Copy link
Contributor

I've noticed a bug lately where I have to refresh the page too make that panel work

@jsimmonds2
Copy link
Contributor Author

Yes, same. That toggle slider seems to have some busy code underneath it !

@jsimmonds2
Copy link
Contributor Author

jsimmonds2 commented Nov 21, 2019

Finally got my Firmware Debug logs :)

  • Built a farmduino_k14.hex with debug logging enabled.
  • Forced firmware_debug_log to true in FbosConfig using
FarmbotCore.Asset.Private.mark_dirty! FarmbotCore.Asset.fbos_config(), %{firmware_debug_log: true}

Surely arduino_debug_messages is now/nearly obsolete/superseded ?

P.s. Also noticed that the firmware_debug_log config setting is not a member of BotState (?)

@ConnorRigby
Copy link
Contributor

i believe arduino_debug_messages and firmware_debug_log are the same thing. There may be a bug where botstate.arduino_debug_messages is not updated when firmware_debug_log is updated and vice versa.

@ConnorRigby
Copy link
Contributor

oh actually, i see that both of those values are real values, not one as a mirror of the other. i'll have to cross reference which one is supposed to be used for this feature.

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 a pull request may close this issue.

3 participants