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

donexon varmo TZ pro thermostat - sensor type 0x17 Water temp indication #1818

Open
markus7h opened this issue Oct 27, 2022 · 7 comments
Open

Comments

@markus7h
Copy link

The donexon thermostat is a danfoss derivate which include additional sensor information over the the original and other derivates like devolo etc.

under the commend class COMMAND_CLASS_SENSOR_MULTILEVEL with sensor type 0x17 the water temp value can be read from the device

please include device specific detection for the donexon with additional water sensor channel
device type: slave
manufacturor id: 0x002
product type: 0x5fff
product id: 0xa010

@cdjackson
Copy link
Collaborator

As far as I can tell this is already in the database.

https://opensmarthouse.org/zwavedatabase/295

@markus7h
Copy link
Author

you missed out, what I mentioned in my text. the devolo and others that are in the database ONLY have one temp sensor. this leads to a working device but with the lack of one temp channel. I do not know how to get an additional channel working if it does not come from the binding, but if there is a way, please let me know.

@apella12
Copy link
Contributor

So I think two things have to happen for this to work

  1. sensor_watertemperature needs to be added to the DB drop down channel options. The following is already in the bindingWATER_TEMPERATURE(23, "WaterTemperature"), , so I do not think anything needs to be done there? OR is there some other dropdown option that will pick this up?
  2. @markus7h will need to add a new channel under the Sensor multilevel CC with that option in the community-maintained DB. Read the blog to obtain write access.

PS- I accidently modified the device while investigating the existing drop-down options but did get it back & marked for review. Sorry

@markus7h
Copy link
Author

I did open a ticket to be able to make the suggested changes, let see how it turns out

@apella12
Copy link
Contributor

apella12 commented Nov 26, 2022

I realized on my afternoon run that my suggested dropdown might not work (or at least might have the temperature quantity). I'm thinking a dropdown of water_temperature[sensor_temperature] might be an alternative based on the other entries. Chris will be the only one that knows.

In support of all this, can you get a debug level file of the device sending that command. In particular the hex string. Look for this section 31 05 17 ?2 ?3, etc. 31 is Multilevel Sensor; 5 is Report; 17 is the water temperature ID; ?2 likely provides the size/scale/precision of the remaining Bytes; ?3, etc would be the value. Also what is a typical water temperature for one of these radiators? For instance 31 05 17 22 01 B8 would be 44.0 C

@apella12
Copy link
Contributor

apella12 commented Dec 4, 2022

Somewhat related question (for my education). Relating to creating channels, would the dropdown list of channel types over-ride any entry in the Config (type=) box during the DB XML creation process? For example if channel was sensor_dewpoint could the type entry be type=WATER_TEMPERATURE or would it get overwritten to type=DEWPOINT. I'm wondering if there is a workaround for some of the more obscure sensors rather than having to add them to the drop down list each time a new one is needed.

Also thought maybe adding generic sensor_customdecimal on the dropdown could allow for obscure sensors on a more general basis. Folks could pull the type out of the XML

@cdjackson
Copy link
Collaborator

I did open a ticket to be able to make the suggested changes

I don't remember the ticket in question (I get a lot of them) but I guess this was actioned and you now have access.

would the dropdown list of channel types over-ride any entry in the Config (type=) box

No - they are completely independent.

Also thought maybe adding generic sensor_customdecimal on the dropdown could allow for obscure sensors on a more general basis.

As far as I'm aware, sensor_customdecimal is not a channel type, so it's not as simple as adding it to the dropdown. It will also require binding changes to the code to actually implement.

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

No branches or pull requests

3 participants