-
Notifications
You must be signed in to change notification settings - Fork 59
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
The documented specification shows an invalid example for Device Attributes #203
Comments
I've noticed something else in this example that appears to be invalid. The homie/super-car/$nodes example sets that attribute to "wheels,engine,lights[]". It is not specifically called out, but I believe these values are supposed to be the ID of the nodes that are supported. "lights[]" is not a valid ID according to section 2.1. |
fixed the example for $extensions. I cannot find the "lights[]" reference. That was part of the convention prior to v4 |
lights[] used to be a valid id and represented an array which got removed in v4. So its possible the examples where missed during updating. |
I looked and didn't find any. @cmoore42 if you could point to where the "lights []" reference is I'll fix. |
Homie specification online at https://homieiot.github.io/specification/
Section 7.1.1.
At the end of the section, just before 7.1.2, the following example is given:
homie/super-car/$homie → "2.1.0"
homie/super-car/$name → "Super car"
homie/super-car/$nodes → "wheels,engine,lights[]"
homie/super-car/$implementation → "esp8266"
homie/super-car/$state → "ready"
The value for the $nodes attribute is not valid under this version of the spec.
From: Michael Cumming <notifications@github.com>
Sent: Tuesday, May 12, 2020 12:01 PM
To: homieiot/convention <convention@noreply.github.com>
Cc: cmoore42 <chrismoore@surewest.net>; Mention <mention@noreply.github.com>
Subject: Re: [homieiot/convention] The documented specification shows an invalid example for Device Attributes (#203)
I looked and didn't find any. @cmoore42 <https://github.com/cmoore42> if you could point to where the "lights []" reference is I'll fix.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#203 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABBCKDW65C6JVJHXAAOH2CLRRGMHXANCNFSM4M5LWTDA> . <https://github.com/notifications/beacon/ABBCKDXCT5D4NY5SKGBPQR3RRGMHXA5CNFSM4M5LWTDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEVTWB7Q.gif>
|
@Thalhammer not sure how to fix this, the convention.md file has been fixed but those changes have not propagated to the homie website. |
Looks like the website is updated by travis and in fact if you look here, it has. But we cant simply do that as the master branch already includes other changes that should not be included in 4.0.0. |
@Thalhammer agree, we need to a process to manage typo/verbiage issues and protocol changes. Happy to do that any way you want. |
I just ran into this snafoo too. I was trying to figure out what this |
This apparently was already fixed in 12453d8 in 2019. Unfortunately, that hasn't been deployed yet it seems |
I don’t see a formal pull request for the change
…________________________________
From: Sören Beye ***@***.***>
Sent: Tuesday, January 4, 2022 5:56:12 AM
To: homieiot/convention ***@***.***>
Cc: Michael Cumming ***@***.***>; Comment ***@***.***>
Subject: Re: [homieiot/convention] The documented specification shows an invalid example for Device Attributes (#203)
This apparently was already fixed in 12453d8<12453d8> in 2019.
Unfortunately, that hasn't been deployed yet it seems
—
Reply to this email directly, view it on GitHub<#203 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAKEYCBDXA6VXAUR3UXXY3DUULG4ZANCNFSM4M5LWTDA>.
Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you commented.Message ID: ***@***.***>
|
@mjcumming The PR for it was #177. The change is there, it's just not deployed properly everywhere. |
afaict the change were deployed. So closing this. please reopen if something is still missing. |
Section 7.1.1 in the spec says, "The following device attributes are mandatory and MUST be send, even if it is just an empty string". It then lists the mandatory Device attributes: $homie, $name, $state, $nodes, and $extensions.
This is followed by an example for a "super-car". However, that example does not contain a $extensions topic, which the specification says is mandatory.
The text was updated successfully, but these errors were encountered: