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 support for LV600s #112
Conversation
if mode.lower() not in ['sleep', 'auto', 'humidity']: | ||
logger.warning('Invalid humidity mode used (sleep, auto, or ' | ||
'humidity)- %s', | ||
mode) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a difference between auto and humidity? Are there three modes on all devices? I only see auto and sleep mode on the other humidifiers. May have to put feature logic to adjust mode settings per device.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't actually tried auto
. I based this PR off of what I was see in my captures. Agreed on adding the feature logic.
TBH, with all of the devices I am seeing in their app, this pattern is going to be outgrown pretty quick. :)
{
"traceId": "[TRACE_ID]",
"payload": {
"method": "setHumidityMode",
"data": {
"mode": "humidity"
},
"source": "APP"
},
"token": "[TOKEN]",
"appVersion": "VeSync 3.2.6 build6",
"configModule": "WFON_AHM_LUH-A602S-WUS_US",
"method": "bypassV2",
"cid": "[CID]",
"debugMode": false,
"deviceRegion": "US",
"phoneBrand": "iPhone 12 Pro",
"userCountryCode": "US",
"acceptLanguage": "en",
"accountID": "[ACCOUNTID]",
"timeZone": "America\\/Chicago",
"phoneOS": "iOS 15.4"
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
auto
didn't work for me. I changed my PR to simplify the interface. The caller won't have to know the distinction with this approach.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I can confirm with my testing that auto
seems to have been renamed to humidity
for the LV600S.
looking forward to seeing this PR soon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will have to make a new class or restructure logic. If you enjoy painful activities, building a test would be a great idea 😉
Any update on this? |
I am in the process of refactoring the library to make it easier to add
devices. Should be ready in a day or two.
…On Tue, Mar 1, 2022 at 12:15 PM Andrew Noblet ***@***.***> wrote:
Any update on this?
—
Reply to this email directly, view it on GitHub
<#112 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB6JJBSXX4TWHMANOLUSKNLU5ZGBPANCNFSM5NYPZ7VA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Sorry, just getting back to this. Do you want me to provide tests for this, or would you prefer to just wait for the new release? |
Any word on when this will be merged and be available in new version? |
I have a branch ready to test, I incorporated all of your changes except the readme file. I will update that in the next day or so when hopefully testing works - https://github.com/webdjoe/pyvesync/tree/air-refactor |
I don't see my humidifier listed with this branch. |
Can you add |
I fairly novice at this. Which file would I put that in? This is what I'm trying:
Only my air purifiers are returned. |
@anoblet I just added a debug feature so when you instantiate the class you can use |
I tried the air-refactor code and it fails for me
|
@brianhealey @adityapavan18 @anoblet Can you give the updated #121 branch a shot. It should be working now |
@webdjoe - no errors like before while running manager.update But when I try to display the details using
Information doesn't show up properly for Air Purifier
|
Here are my new logs: https://gist.github.com/anoblet/424a1ad88ba53e93218511246859b03c |
This should be all set, releasing pypi package now. Let me know if there are still any issues |
Will this be added to Home Assistant? |
@anoblet you would have to submit a PR to HA repo to update the pyvesync package requirements to 2.0.0 unless you build from scratch |
You would have to submit a PR to HA to change the package version in
requirements or build from the source. The latest version is 2.0.0
…On Tue, Mar 22, 2022 at 8:47 PM Andrew Noblet ***@***.***> wrote:
Will this be added to Home Assistant?
—
Reply to this email directly, view it on GitHub
<#112 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB6JJBX3X52ZZ3Y3A4YAWPTVBJS2RANCNFSM5NYPZ7VA>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
No description provided.