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] Why chip tools still send InteractionModelRevision = 1 instead of 10? #29296

Closed
Apollon77 opened this issue Sep 15, 2023 · 4 comments
Closed
Labels
bug Something isn't working needs triage

Comments

@Apollon77
Copy link
Contributor

Reproduction steps

When you look at the commands that chip tool and also the chip apps send all interaction messages are send with

InteractionModelRevision = 1

The Matter spec 1.0 had set this revision number to 10, so why this is not used? versions 1..9 are some "versions of the Zigbee Cluster Library chapter 2 which combined the interac­ tion model with encoding" ...

Bug prevalence

always

GitHub hash of the SDK that was being used

any

Platform

other

Platform Version(s)

No response

Anything else?

No response

@Apollon77 Apollon77 added bug Something isn't working needs triage labels Sep 15, 2023
@Apollon77 Apollon77 changed the title [BUG] WHy chip tools still send InteractionModelRevision = 1 instead of 10? [BUG] Why chip tools still send InteractionModelRevision = 1 instead of 10? Sep 15, 2023
@bzbarsky-apple
Copy link
Contributor

@Apollon77 Please list the actual github hash involved, not "any". This was fixed in #28645 over a month ago.

@bzbarsky-apple
Copy link
Contributor

@Apollon77 That said, if you are in fact seeing that on tip, please reopen as needed. Also, note that the spec has been updated to make it clear that the revisions the 1.0 SDK sent did not match the 1.0 spec. How things got there... that's a separate story.

@Apollon77
Copy link
Contributor Author

@bzbarsky-apple sorry for it ... I thought I updated but in fact did not ... basically I run (again ... had that before, learned and forgot ;-) ) into #29299 ... so now I tried to update and run into #29300 because newest builds of chip tool do not have tests in any more and no docs were updated :-(

@Apollon77
Copy link
Contributor Author

But yes now after I managed to somehow get it running the reported revision is 11 (most likely Matter 1.2 coming .. soooo) is fine now. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage
Projects
None yet
Development

No branches or pull requests

2 participants