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

Pb error msgs #584

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Pb error msgs #584

wants to merge 16 commits into from

Conversation

tyeth
Copy link
Contributor

@tyeth tyeth commented May 8, 2024

Adds extra logging on encoding errors for protobuf/nanopb.

@tyeth
Copy link
Contributor Author

tyeth commented May 17, 2024

Feedback welcomed, just threw this together for that PyPortal Titano user, ran it and saw data flow normally but never saw an error so it's utility remains untested.

@brentru
Copy link
Member

brentru commented May 17, 2024

@tyeth Have you tested this on physical HW yet and played around with it?

@tyeth
Copy link
Contributor Author

tyeth commented May 17, 2024 via email

@tyeth
Copy link
Contributor Author

tyeth commented May 23, 2024

Skipping this release (beta83), will be reviewed in context (PyPortal Titano) next week, then make it into 84.

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.

None yet

2 participants