-
Notifications
You must be signed in to change notification settings - Fork 510
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
Suggested correction to intent response for CO2 sensor #2350
Comments
There are no binary sensors associated with carbon dioxide. |
Sorry if I wasn't clear in my description. The inconsistency is in the Intents (questions/responses) to control the sensors through Assist not the binary sensors themselves. The questions and device class refer to "carbon monoxide" yet the answers refer to CO2 (which is carbon DI-oxide). The references to CO2 in the responses should refer to "carbon monoxide" to be consistent. For example in the English translation, I am referring to lines 107-149 of: https://github.com/home-assistant/intents/blob/main/tests/en/binary_sensor_HassGetState.yaml Below is an example from that yaml file:
Thanks for reviewing and considering the suggestion. |
Thanks for the clarification. This is a typo in the name of the sensor used for the tests. It's not critical at all, but you can fix it. https://github.com/home-assistant/intents/blob/main/tests/en/_fixtures.yaml#L177 |
Re: intents/tests/en/binary.sensor.HassGetState.yaml
When reviewing the possible phrases, I noticed there are multiple questions and responses that incorrectly use the term "carbon monoxide" to refer to the state of a CO2 sensor.
While I can see a use case for also testing a Carbon Monoxide sensor, CO2 is the chemical formula for Carbon Dioxide, a measure of air quality, not Carbon Monoxide (CO) the more deadly variant from combustion.
Suggest all current references to "monoxide" in this intent configuration be changed to refer to "dioxide" and recommend condider adding separate intents for a CO binary sensor. The issue also exists in other languages. Note: May have implications for the "device_class" as well.
The text was updated successfully, but these errors were encountered: