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

added new command class ManufacturerProprietary for Fibaro FGRM-222 (part 2) #1204

Merged
merged 7 commits into from Apr 30, 2017

Conversation

ChristianKuehnel
Copy link
Contributor

@ChristianKuehnel ChristianKuehnel commented Apr 9, 2017

As discussed some time ago in the forum: The Fibaro FGRM-222 Roller Shutter 2 is using the COMMAND_CLASS_MANUFACTURER_PROPRIETARY to indicate the tilt angle of the slats. This is only transmitted when activating the venetian blind mode and setting the report type to "Fibaro Command class".

@Fishwaldo
As I messed up my branch completely, it was easier to create a new, clean one. So this is the continuation of pull request #1203

@ChristianKuehnel
Copy link
Contributor Author

As I did not get any feedback on the design/architecture, I implemented the remaining features in the same way.

I tested these successfully with my devices:

  • handling of received messages is working
  • requesting the values is working
  • setting the position and tile of the venetian blinds is also working

@Fishwaldo Fishwaldo self-assigned this Apr 30, 2017
@Fishwaldo Fishwaldo added this to the Version 1.6 milestone Apr 30, 2017
@Fishwaldo Fishwaldo merged commit 9270fd9 into OpenZWave:Dev Apr 30, 2017
@Fishwaldo
Copy link
Member

Thanks

@ChristianKuehnel
Copy link
Contributor Author

Hi @Fishwaldo,

what do I have to do to get this merged to master? I would like to use it in Home Assistant and they are using the python-openzwave library which seems to be using some snapshots from master.
https://github.com/OpenZWave/python-openzwave/tree/master/archives

@andre-richter
Copy link

Hi @nechry,

I tried to follow all kinds of resources to understand why this change is not being merged into master. From what I understand, it is sitting in the DEV branch that is supposed to be merged into master. Unfortunately, this seems to be dormant since over a year now.

Same as @ChristianKuehnel, I would be super happy to see this in main, because like him, I am using Home Assistant with lots of Fibaro Roller shutters and would really like to get control over the slat angle. I'm sure there's lots of other users that would profit from this as well.

Kindly let us know if there is anything that we can help you with to accelerate getting this merged.

Thanks a lot,
Andre

@tameike000
Copy link

@ChristianKuehnel Do I read it correctly your fix is still stitting in the dev branch and is a target for 1.6 (currently we are still on 1.4?)

How did you integrate it into HA? Don't mind to install the dev branch. I really want that fix.

Thanks.
M

@petergebruers
Copy link
Collaborator

The FGRM-222 Venetian Blind mode is supported in OpenZWave 1.6 and is working, you can test with "ozwcp". I know this because I own one. But HASS is still using a fork of 1.4.

I opened a PR to add Venetian blind support to Domoticz beta, but it was not accepted yet because it requires changes to Domoticz device handling.

So I am not sure which software actually supports Venetian blind mode right now...

@tameike000
Copy link

@petergebruers Thanks for the heads up. I will raise it in HA!

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

Successfully merging this pull request may close these issues.

None yet

5 participants