-
Notifications
You must be signed in to change notification settings - Fork 490
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
Created configuration for Orison Chanfok Neo #2129
Conversation
I created the config for this "fandelier", which is also found on some shops sold as "Orison Ceiling Fan with Lights- 23 in Smart Ceiling Fans with Alexa/Google Assistant/App Control Color Changing LED-RGB Back Ambient Light for Living Room Bedroom". Everything works perfectly!
I believe that I have this same Orison Chanfok Neo ceiling fan light. Mine is Product ID: zfwgeelmqcypcdgh. Does your fan have two LEDs? RGB facing the ceiling and white facing down??? I've been working on some additional DPs... 36 = mix_light_scene (I've got this one setting both the upper RGB LEDs and the lower white LED ring)
|
@@ -0,0 +1,102 @@ | |||
name: Orison Chanfok Neo | |||
products: | |||
- id: eba992532f93bb1c009bjw |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a device id rather than a product id. Is the product id perhaps zfwgeelmqcypcdgh
as per the previous comment?
@@ -0,0 +1,102 @@ | |||
name: Orison Chanfok Neo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This name should be generic, such as Ceiling fan with light
type: string | ||
name: direction | ||
- id: 103 | ||
type: integer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be a sensor of class duration if you know the units (min or s).
Based on the name, it should probably be coupled with a method of setting the timer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does your fan have two LEDs? RGB facing the ceiling and white facing down???
Yes, and with this config I can control only the main light. This config is not fully implemented but at least the big things work (fan and main light)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reply does not seem to match with the comment. I think you are actually replying to another comment on the main PR by another user.
Here's how I made the "mix_rgbcw" (DP 51) work to set both the upper RGB LED and the lower white LED... For a red upper light and white (6500K) lower light, both at 100% brightness, send The hex bits encoding is as follows... 00-01: Mode (4 = off, 5 = white only, 6 = colour only, 7 = both)
This works to pre-define some color combinations similar to a scene. However, I would like to utilize the color and temp pickers in HA to build this combination string on-the-fly. #struggling |
Changes from review. Incorporated some changes from @RobDehnert mix_rgbcw left for now, as there is another issue for this type of dp on another device where I hope to make a better decoding of the values. PR make-all#2129
Avoid long lines PR make-all#2129
Changes from review. Incorporated some changes from @RobDehnert mix_rgbcw left for now, as there is another issue for this type of dp on another device where I hope to make a better decoding of the values. PR #2129
I created the config for this "fandelier", which is also found on some shops sold as "Orison Ceiling Fan with Lights- 23 in Smart Ceiling Fans with Alexa/Google Assistant/App Control Color Changing LED-RGB Back Ambient Light for Living Room Bedroom".
Everything works perfectly!