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

add sensors to curtain motor #213

Closed
vnkorol opened this issue Jul 25, 2022 · 11 comments
Closed

add sensors to curtain motor #213

vnkorol opened this issue Jul 25, 2022 · 11 comments

Comments

@vnkorol
Copy link

vnkorol commented Jul 25, 2022

"product_id": "3r8gc33pnqsxfe1g",
"product_name": "Blinds Controller",

To visualize position and check status of curtain, please add two sensors:
DP 2 - sensor position set - unit '%'
DP 3 - sensor position get - unit '%'

@make-all
Copy link
Owner

Which configuration is this modification for?

@make-all make-all added awaiting feedback Needs more information device variant Variant of a supported device labels Jul 25, 2022
@vnkorol
Copy link
Author

vnkorol commented Jul 25, 2022

  1. when I send command to open at 50% I would like to check real position
  2. At lovelace ganel it is more useful to show "get_position" instead of "current_position" from attributes. AFAIK in attributes there is set_position value.

@make-all
Copy link
Owner

I think you misunderstood my question. I mean which config file are you requesting these sensors be added to?

@vnkorol
Copy link
Author

vnkorol commented Jul 26, 2022

To configuration of curtain controller. dp_id 2 and 3.

{
"result": {
"category": "cl",
"functions": [
{
"code": "control",
"dp_id": 1,
"type": "Enum",
"values": "{"range":["open","stop","close"]}"
},
{
"code": "percent_control",
"dp_id": 2,
"type": "Integer",
"values": "{"unit":"%","min":0,"max":100,"scale":0,"step":1}"
},
{
"code": "control_back",
"dp_id": 5,
"type": "Boolean",
"values": "{}"
},
{
"code": "countdown",
"dp_id": 8,
"type": "Enum",
"values": "{"range":["cancel","1","2","3","4"]}"
}
],
"status": [
{
"code": "control",
"dp_id": 1,
"type": "Enum",
"values": "{"range":["open","stop","close"]}"
},
{
"code": "percent_control",
"dp_id": 2,
"type": "Integer",
"values": "{"unit":"%","min":0,"max":100,"scale":0,"step":1}"
},
{
"code": "percent_state",
"dp_id": 3,
"type": "Integer",
"values": "{"unit":"%","min":0,"max":100,"scale":0,"step":1}"
},
{
"code": "control_back",
"dp_id": 5,
"type": "Boolean",
"values": "{}"
},
{
"code": "work_state",
"dp_id": 7,
"type": "Enum",
"values": "{"range":["opening","closing"]}"
},
{
"code": "countdown",
"dp_id": 8,
"type": "Enum",
"values": "{"range":["cancel","1","2","3","4"]}"
},
{
"code": "countdown_left",
"dp_id": 9,
"type": "Integer",
"values": "{"unit":"s","min":0,"max":86400,"scale":0,"step":1}"
},
{
"code": "time_total",
"dp_id": 11,
"type": "Integer",
"values": "{"unit":"ms","min":0,"max":120000,"scale":0,"step":1}"
}
]
},
"success": true,
"t": 1658807343754,
"tid": "e435cb7d0c9511ed83de62023b21e71a"
}

@make-all make-all added new device Unsupported device and removed device variant Variant of a supported device labels Jul 26, 2022
@make-all
Copy link
Owner

This appears to be a completely new device, not a minor addition to an existing config. Can you please give some details about the device (brand, model etc) to help with naming the config file.

@vnkorol
Copy link
Author

vnkorol commented Jul 27, 2022

@make-all make-all removed the awaiting feedback Needs more information label Aug 2, 2022
make-all added a commit that referenced this issue Aug 3, 2022
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release labels Aug 3, 2022
@vnkorol
Copy link
Author

vnkorol commented Aug 4, 2022

Now I waiting for release, right?

@make-all
Copy link
Owner

make-all commented Aug 4, 2022

Or you can try it right away by selecting "Redownload" from the HACS page for this integration, and select "main" as the version to download. It will pop up a notification that 0.8.1 is available to upgrade to, but you can ignore that until 0.8.2 comes out.

@vnkorol
Copy link
Author

vnkorol commented Aug 4, 2022

Device is detected correctly.
image

But there is no sensors - position_get (dp_id 3) and position_set (dp_id 2)
image

@make-all
Copy link
Owner

make-all commented Aug 4, 2022

These are not separate sensors, they are part of the main cover entity.

@vnkorol
Copy link
Author

vnkorol commented Aug 4, 2022

It works correctly! Thanks. DP_ID 3 i can get from current_position attribute. Thanks!

@vnkorol vnkorol closed this as completed Aug 4, 2022
@make-all make-all removed awaiting confirmation Wating for confirmation the issue is solved new device Unsupported device unreleased Will be in next release labels Aug 4, 2022
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

2 participants