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

Allergy Mode is missing #144

Open
eljarno opened this issue Mar 23, 2024 · 7 comments
Open

Allergy Mode is missing #144

eljarno opened this issue Mar 23, 2024 · 7 comments
Labels
question Further information is requested

Comments

@eljarno
Copy link

eljarno commented Mar 23, 2024

Right now I cannot change the Allergy Night Mode. When I change it within the App the mode is set to "AS" within the iobroker objects. But when I try to change it via javascript to "AS" nothing happens. I guess the mode can be read but node be changed.

It would be great if you change the Adapter and add the missing mode.

Regards
Jan

@Apollon77
Copy link
Member

Please provide a log - is it set in the objects to "AS" as string or is it shown as a selection there? Can you set in the onbkjects directly but just not via Javascript? Please provide an info

@eljarno
Copy link
Author

eljarno commented Mar 26, 2024

It ist in the objects to "AS" as string. It not shown in the selection. The selection offers an "allergen" option but if I select that one, it has no effect. By the way half of the selection options has no effect at all.

Find attached the screenshot of the object string.
iobroker philips air

@Ralf31337
Copy link

I have exactly the same issue. Can't set it via Objects in IOBroker, because "AS" is not available as a valid value in the drop-down.

In the first two rows, you see the values that come from the device if I set it to allergy sleep via Philips app:
grafik

If I send the string "AS" it does not work. For "turbo" and "automode" it does work.

@mcm1957
Copy link
Member

mcm1957 commented Dec 1, 2024

This issue does not contain version infos and log

Please add version infos
adapter
js-controller
admin
node
o/s

and attach a complete log with loglevel set to DEBUG showing the actions araound setting state 'mode'.

@mcm1957 mcm1957 added the question Further information is requested label Dec 1, 2024
@Ralf31337
Copy link

Adapter version: v1.1.0
Controller version: v6.17.14

That's the WARN I get, when setting it to "AS" via node-red:
  | warn | Could not control mode: Invalid option for mode: AS. Supported only: {"P":"auto","A":"allergen","S":"sleep","M":"manual","B":"bacteria","N":"night","T":"turbo","AG":"automode","GT":"gentle"}

That's what I receive in DEBUG, when setting to Allergy Sleep in the app:
State change: {"val":"AS","ack":true,"ts":1733162331433,"q":0,"from":"system.adapter.philips-air.0","user":"system.user.admin","lc":1733162255368}

I think it is not allowed from what I can see in the code:

mode: {name: 'mode', options: {'P': 'auto', 'A': 'allergen', 'S': 'sleep', 'M': 'manual', 'B': 'bacteria', 'N': 'night', 'T': 'turbo', 'AG': 'automode', 'GT': 'gentle'}, control: true},

There is an allergen available, but the AC4236 has a different name for it. Does it make sense to add all variants? Or make the adapter device specific? Philips seems to be really inconsistent here. :-(

@mcm1957
Copy link
Member

mcm1957 commented Dec 2, 2024

Yes - I already considered to ask wether this functionality ever worked as I did not see an AS entry too.

As you already detected this code, I would suggest that you - AS AN EXCEPTION FOR FIXING - and if you are a little bit familar with some editor try to change the code an insert some 'AS': 'allergen' or 'AS': 'allergenmode' entry. I do not have the equipment to test anything and mosg likely I do not have time to provide test releases but I can offer to create a new release if at least a testable fix is available.

Of course an PR would be welcom most if you are familar with GitHub.

@Ralf31337
Copy link

I will create a PR and do the modifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants