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

Current KNX does not correct import or gui does not use dpt5.001 #123715

Closed
pos-ei-don opened this issue Aug 12, 2024 · 7 comments · Fixed by #123778
Closed

Current KNX does not correct import or gui does not use dpt5.001 #123715

pos-ei-don opened this issue Aug 12, 2024 · 7 comments · Fixed by #123778
Assignees

Comments

@pos-ei-don
Copy link

pos-ei-don commented Aug 12, 2024

The problem

I use dpt5.001 vor Tunable White.

- name: "abc"
  address: "0/1/229" 
  brightness_address: 0/1/234
  brightness_state_address: 3/2/18
  color_temperature_address: 0/1/233
  color_temperature_state_address: 0/1/244
  color_temperature_mode: relative

worked in yaml.

When I not use the new gui to create a LED entity,
it does not work!
I did reimport my KNX project!

Here you see the Group Monitor.
In the upper part you see the Device when I use the YAML-Defined Entity,
in the lower part you see what happens when i use the GUI defined Entity.
It sends values in the wrong dpt.

Edit: Brightness is sent correct as DPT5.001

image

Here you see the gui configuration
image

What version of Home Assistant Core has the issue?

2024.08.1

What was the last working version of Home Assistant Core?

What type of installation are you running?

Home Assistant OS

Integration causing the issue

knx

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@pos-ei-don
Copy link
Author

Hey there @Julius2342, @farmio, @marvin-w, mind taking a look at this issue as it has been labeled with an integration (knx) you are listed as a code owner for? Thanks!

@home-assistant
Copy link

Hey there @Julius2342, @farmio, @marvin-w, mind taking a look at this issue as it has been labeled with an integration (knx) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of knx can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign knx Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


knx documentation
knx source
(message by IssueLinks)

@pos-ei-don pos-ei-don changed the title Current KNX does not correct import dpt5.001 Current KNX does not correct import or guidoes not use dpt5.001 Aug 12, 2024
@pos-ei-don pos-ei-don changed the title Current KNX does not correct import or guidoes not use dpt5.001 Current KNX does not correct import or gui does not use dpt5.001 Aug 12, 2024
@farmio
Copy link
Contributor

farmio commented Aug 13, 2024

Thanks for filing this issue 👍

@userwithoutpassword
Copy link

Mh when dimming to the minimum Level 0x01 (0%) HA "thinks" the Light is off, but it is not. Is there a way that we could only interpret 0x00 as "off" and 0x01 for the minimum dimming level?

@farmio
Copy link
Contributor

farmio commented Aug 14, 2024

@userwithoutpassword I don't think this is related to this issue. Please create a separate one (if you like here https://github.com/XKNX/knx-integration ) for this.

@pos-ei-don
Copy link
Author

0x01

+1. I have actors where 0x01 is darker than 0x02, so deximals 0-255 would better fit than 0-100%.
So to be able to set minimum brightness would be great. If you Tell me the link of your issue, I try to upvote.

@pos-ei-don
Copy link
Author

Tested New Update, works as expected. Thank you!!

@github-actions github-actions bot locked and limited conversation to collaborators Sep 17, 2024
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants