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

Capture device debug content and log it #117

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

Conversation

benbrown
Copy link
Contributor

Fixes #101 - this captures the ascii text that comes between packets and logs it as a trace.

I also added some comments in to explain what the protocol handler is doing.

…d some comments to transformer to demystify protocol handling.
@Hunter275
Copy link
Member

Hunter275 commented Nov 24, 2024

Build is failing due to version not being bumped, we really should have the versioning be done by the Action itself.

@benbrown
Copy link
Contributor Author

@Hunter275 I did not want to be presumptuous about that but happy to do so if that is protocol...

@benbrown
Copy link
Contributor Author

BTW, in the web client this seems to result in timeout errors logged for the heartbeart. I do not see these in the node serial version...

Seems like there are some events sent to the radio that do not ever get the ack expected?

@Hunter275
Copy link
Member

@Hunter275 I did not want to be presumptuous about that but happy to do so if that is protocol...

We'll probably wrap several of your PRs (and others) into a single version bump so not necessary

@benbrown
Copy link
Contributor Author

@Hunter275 yeah ok. seems right.

I am eager to help, let me know what I can do

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