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

Update Log reader for Bridge Controller (0xA8) #44

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

Conversation

apella12
Copy link

@apella12 apella12 commented Apr 8, 2023

Added Bridge Controller Byte code (A8). Sensor Binary and Sensor Multilevel changes for clarity.

Bridge viewer PR;
viewer-bridge
Sensor multi with bridge (0xA8) PR
viewer-multisensor-new-bridge
sensor multi with application (0x04) PR
viewer-multisensor-new
sensor multi -old with application (0x04)
viewer-multisensor-old
sensor binary with bridge PR
viewer-binary-new-bridge
sensor binary with application PR
viewer-binary-new
sensor binary with application-old
viewer-binary-old

apella12 added 2 commits April 8, 2023 09:56
Added Bridge Controller Byte code (A8). Sensor Binary and Sensor Multilevel changes for clarity.

Signed-off-by: Bob Eckhoff <[email protected]>
Recognize PR is on old version so is not helpful, however upon looking at several debug files I have made some changes for clarity (in use for me locally). Also made additional change to the bridge (A8) because it is also longer at then end,  so the command class processing was getting messed up in some cases.

Signed-off-by: Bob Eckhoff <[email protected]>
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.

1 participant