zpieslak / mobilus-client-home-assistant

Home Assistant integration for Mobilus Cosmo GTW
Apache License 2.0
2 stars 0 forks source link

doesn't open shutters controlled by the mobilus C-MR module #9

Closed danielkedzior closed 2 days ago

danielkedzior commented 3 days ago

Integration doesn't open roller shutters controlled by the mobilus C-MR module. It reads their position (open/closed), but shutters do not react after the open/close signal.

zpieslak commented 3 days ago

Hey @danielkedzior,

Thanks for the report. Taking your answer from the forum (reference).

I may have idea how to fix it. The current implementation assumes that shutters can receive percentage command and this is how they are controlled in HA (open is de-facto setting percentage to 100% and close to 0%). But maybe this is now always the case (as you shown in the forum). Fortunately there are also command UP and DOWN and I think this is how official app uses it.

Anyway this shouldn't be hard and I will try to fix it in spare moment.

danielkedzior commented 3 days ago

Hello, thank you for your work on this subject. This appears to be the case, as I am unable to control them in any other way. As an interesting aside, my friend has roller shutters made by krispol, with mobilus drives. None of them have any control of the opening level, you can only open, close or stop them while they are running. Whether by remote control or via a app

zpieslak commented 3 days ago

@danielkedzior Hmm, maybe the Krispol hardware is the same as you have in C-MR, who knows :) Anyway, please check the branch in the mentioned pull request and let know if this works now for you. If so, I will merge the pull request. Thanks!

danielkedzior commented 2 days ago

Hi @zpieslak, I have checked, it works properly. It works with standard shutters and also with the ones controlled by C-MR. Thanks!

zpieslak commented 2 days ago

@danielkedzior Great! Thanks for testing. I will merge the pull request then

Jasbi90 commented 17 hours ago

Hello, Thank you for your work on this plugin.

I also have Mobilus shutters (COSMO) without the percentage setting. When I add these attributes, I don't see a slider (which I don't need).

image

In the Mobilus HACS plugin, I see the following with this attribute: image image

Would it be possible to implement something from the app-6a66bade0jd.js file of Mobilus?

function() { "use strict"; function e(e) { var t = {} , i = { 1: { deviceName: "SENSO", deviceType: "updownstopslider", deviceDefaultIconId: 1 }, 2: { deviceName: "COSMO", deviceType: "updownstop", deviceDefaultIconId: 1 }, 3: { deviceName: "CMR", deviceType: "updownstop", deviceDefaultIconId: 1 }, 4: { deviceName: "CGR", deviceType: "onebutton", deviceDefaultIconId: 1 }, 5: { deviceName: "SWITCH", deviceType: "toggle", deviceDefaultIconId: 5 }, 6: { deviceName: "SWITCH_NP", deviceType: "toggle", deviceDefaultIconId: 5 }, 7: { deviceName: "COSMO_CZR", deviceType: "updownstopdownandslider", deviceDefaultIconId: 35 }, 8: { deviceName: "COSMO_MZR", deviceType: "updownstopdownandslider", deviceDefaultIconId: 35 }, 9: { deviceName: "SENSO_Z", deviceType: "updownstoptwoslider", deviceDefaultIconId: 35

&

var w = [{ groupText: "toggle", permittedDeviceTypes: [5, 6] }, { groupText: "onebutton", permittedDeviceTypes: [4] }, { groupText: "updownstop", permittedDeviceTypes: [1, 2, 3, 7, 8, 9], moreDetailedGroups: [{ groupText: "updownstopslider", permittedDeviceTypes: [1, 9], ancestorDevices: [2, 3, 7, 8], moreDetailedGroups: [{ groupText: "updownstoptwoslider", permittedDeviceTypes: [9], ancestorDevices: [1, 2, 3, 7, 8] }] }, { groupText: "updownstopdownandslider", permittedDeviceTypes: [7, 8, 9], isSpecialConditionsPassed: function(e) { return void 0 !== e && null !== e && void 0 !== e.icon && null !== e.icon && 35 == e.icon ? !0 : !1 }, ancestorDevices: [1, 2, 3, 9], moreDetailedGroups: [{ groupText: "updownstoptwoslider", permittedDeviceTypes: [9], ancestorDevices: [1, 2, 3, 7, 8] }]

It seems that setting a percentage is not possible for the COSMO and CMR devices, which only support up, down, and stop controls.

EDIT: Thanks. In the cover.py i have now -> # | CoverEntityFeature.SET_POSITION so now it works for me.

zpieslak commented 10 hours ago

@Jasbi90 Thanks for your report! I created a separate issue. Let's continue there.