-
Notifications
You must be signed in to change notification settings - Fork 481
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
Salus PS600 Pipe Temperature Sensor #6871
Comments
Hi, May be you have to create two files under the folder for DDF files on your system The first one with a name like
Then a second file in the same place named
Édit : added some missing lines in second file. Sorry |
Ok, thanks. I'm using deconz in Home Assistant so I'll have to work out where to put these. |
May be this could help #6798 (comment) You'll have to change DDF level too, after your files were created https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/How-to-change-DDF-modes-and-enable-or-disable-DDF-levels |
Hi @BabaIsYou - thanks for the help. I've put your suggested files in the appropriate folder, but deconz doesn't seem to be picking them up and if I try to open in the DDF editor it give an error Failed to Open - any thoughts? |
Yes, very sorry about that, ... I thought I validate it before pasting but seems than copy/paste didn't work as expected (trying to make a DDF on a tablet is something I hate). Well I corrected the DFF above (deleted an unwanted comma after Simply do then same in your copy on your system or change the complete DDF. |
@BabaIsYou - thanks very much - I've managed to get it working now. I found a couple of other typos in the file, I suspect from when I copied it across - all seems good now. Thanks very much. |
Good ! Do you want to make the pull request (PR) to have included in a next release ? |
I would do, just not sure how! |
This time it’s not as simple as just adding a .json to a folder ;-) Github will ask you to commit on master branch then add comments and something like fix #6871 to associate PR to this request. And that’s it. Don’t worry PR will be reviewed and if something went wrong just delete the PR ;-) |
OK - think I've done that right - let's see, thanks for the help. |
Looks Ok, bravo ! ;-) |
I see you did it and the issue is now linked to the PR. Thx |
No problem. Thanks for the help. |
Device
Screenshots
Clusters of the Node
Node Info Panel
Basic
Identify
Diagnostics
Other clusters that are not mentioned above
Power Configuration
Poll Control
Temperature Measurement
IAS Zone
The text was updated successfully, but these errors were encountered: