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

Expand number of airSwingUD modes #16

Open
eckhart opened this issue Oct 11, 2019 · 3 comments
Open

Expand number of airSwingUD modes #16

eckhart opened this issue Oct 11, 2019 · 3 comments

Comments

@eckhart
Copy link

eckhart commented Oct 11, 2019

Apparently, there are more modes than those listed in constants.py for airSwingLR, at least for my device. According to the app the Air swing (Left-Right) was center + 1 to the left.

Traceback (most recent call last): File "/usr/local/bin/pcomfortcloud", line 11, in <module> sys.exit(main()) File "/usr/local/lib/python2.7/dist-packages/pcomfortcloud/__main__.py", line 187, in main print_result( session.get_device(device['id']) ) File "/usr/local/lib/python2.7/dist-packages/pcomfortcloud/session.py", line 239, in get_device 'parameters': self._read_parameters(_json['parameters']) File "/usr/local/lib/python2.7/dist-packages/pcomfortcloud/session.py", line 374, in _read_parameters value['airSwingHorizontal'] = constants.AirSwingLR(parameters['airSwingLR']) File "/usr/lib/python2.7/dist-packages/enum/__init__.py", line 338, in __call__ return cls.__new__(cls, value) File "/usr/lib/python2.7/dist-packages/enum/__init__.py", line 651, in __new__ raise ValueError("%s is not a valid %s" % (value, cls.__name__)) ValueError: 5 is not a valid AirSwingLR

@lostfields
Copy link
Owner

Ok, we have to track down what differs from your model by using dump, but I can see it's hard to track down everything so I can try to make a diff tool or something.

@nimikko
Copy link

nimikko commented Mar 23, 2020

@lostfields Are you sure that the AirSwingLR values are correct as I went them trough on my unit and go the following values:

class AirSwingLR(Enum):
Auto = -1
Left = 1
LeftMid = 5
Mid = 2
RightMid = 4
Right = 0

So they are the same but in reverse order and the one with 3 is 5 at least on my heatpump.

@com-xuonghuynh
Copy link

com-xuonghuynh commented Jun 5, 2020

Some AC model don't have Air swing LR only Up and Down

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

4 participants