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

Print mrp and tcp values in python discover. #9755

Merged

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Sep 16, 2021

Problem

MRP and T flags are not printed in the discovery summary in the python controller

Change overview

Adds the prints

Testing

Tested against lighting app with and without the flags set - both prints look fine.

@mspang
Copy link
Contributor

mspang commented Sep 17, 2021

Why do we have tabs in logs?

@cecille
Copy link
Contributor Author

cecille commented Sep 17, 2021

tabls just line them up nicely. This function is literally used for nothing but printing as part of the python controller output.

@andy31415 andy31415 merged commit cb511ba into project-chip:master Sep 17, 2021
@cecille cecille deleted the txt_prints_in_python_controller branch September 20, 2021 14:52
mleisner pushed a commit to mleisner/connectedhomeip that referenced this pull request Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants