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

VeSync Integration: Support Levoit 6000s is missing #129120

Open
x10 opened this issue Oct 25, 2024 · 21 comments
Open

VeSync Integration: Support Levoit 6000s is missing #129120

x10 opened this issue Oct 25, 2024 · 21 comments

Comments

@x10
Copy link

x10 commented Oct 25, 2024

The problem

I have a Levoit 6000s humidifier and according to the documentation of the integration those are not supported.

Could we add support for this humidifier?

What version of Home Assistant Core has the issue?

core-2024.10.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

VeSync

Link to integration documentation on our website

https://www.home-assistant.io/integrations/vesync/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @markperdue, @webdjoe, @TheGardenMonkey, @cdnninja, mind taking a look at this issue as it has been labeled with an integration (vesync) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of vesync can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign vesync Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


vesync documentation
vesync source
(message by IssueLinks)

@ncd7
Copy link

ncd7 commented Oct 28, 2024

Seconded, came to file the same issue :).

@soulsniper753
Copy link

I have the same issue

@bmos
Copy link

bmos commented Nov 10, 2024

Is there any information/logs/etc I can get from my 6000S to help advance this?

@echowarp
Copy link

Here is some information from mine. The device shows up in the logs, but there seems to be a python error when loading the config.

{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2024.11.1",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.12.4",
    "docker": true,
    "arch": "x86_64",
    "timezone": "America/Chicago",
    "os_name": "Linux",
    "os_version": "6.6.54-haos",
    "supervisor": "2024.11.2",
    "host_os": "Home Assistant OS 13.2",
    "docker_version": "27.2.0",
    "chassis": "vm",
    "run_as_root": true
  },
  "custom_components": {},
  "integration_manifest": {
    "domain": "vesync",
    "name": "VeSync",
    "codeowners": [
      "markperdue",
      "webdjoe",
      "thegardenmonkey",
      "cdnninja"
    ],
    "config_flow": true,
    "documentation": "https://www.home-assistant.io/integrations/vesync",
    "iot_class": "cloud_polling",
    "loggers": [
      "pyvesync"
    ],
    "requirements": [
      "pyvesync==2.1.12"
    ],
    "is_built_in": true,
    "overwrites_built_in": false
  },
  "setup_times": {
    "null": {
      "setup": 1.5821075066924095e-05
    },
    "**REDACTED**": {
      "wait_import_platforms": -0.0026208481285721064,
      "config_entry_setup": 0.7598381408024579
    }
  },
  "data": {
    "vesync": {
      "bulb_count": 0,
      "fan_count": 1,
      "outlets_count": 0,
      "switch_count": 0,
      "timezone": "America/Chicago"
    },
    "devices": {
      "bulbs": [],
      "fans": [
        {
          "manager": "**REDACTED**",
          "device_name": "Superior 6000S Smart Evaporative Humidifier",
          "device_image": "https://image.vesync.com/defaultImages/LEH-S601S-WUS/icon_LEH_S601S_WUS_240.png",
          "cid": "***REDACTED***",
          "connection_status": "online",
          "connection_type": "WiFi+BTOnboarding+BTNotify",
          "device_type": "LEH-S601S-WUS",
          "type": "wifi-air",
          "uuid": "**REDACTED**",
          "config_module": "VS_WFON_AHM_LEH-S601S-WUS_US",
          "mac_id": "**REDACTED**",
          "mode": "humidity",
          "speed": null,
          "extension": null,
          "current_firm_version": null,
          "device_region": "US",
          "pid": null,
          "sub_device_no": null,
          "config": {},
          "device_status": "off",
          "_config_dict": {
            "module": "VeSyncSuperior6000S",
            "models": [
              "LEH-S601S-WUS"
            ],
            "features": [],
            "mist_modes": [
              "auto",
              "humidity",
              "sleep",
              "manual"
            ],
            "mist_levels": [
              1,
              2,
              3,
              4,
              5,
              6,
              7,
              8,
              9
            ]
          },
          "mist_levels": [
            1,
            2,
            3,
            4,
            5,
            6,
            7,
            8,
            9
          ],
          "mist_modes": [
            "auto",
            "humidity",
            "sleep",
            "manual"
          ],
          "details": {
            "humidity": 55,
            "target_humidity": 60,
            "mist_virtual_level": 0,
            "mist_level": 0,
            "water_lacks": 0,
            "water_tank_lifted": 0,
            "filter_life_percentage": 96,
            "temperature": 680,
            "display": 1,
            "drying_mode": {
              "dryingLevel": 1,
              "autoDryingSwitch": 1,
              "dryingState": 1,
              "dryingRemain": 7163
            }
          },
          "_api_modes": [
            "getHumidifierStatus",
            "setSwitch",
            "setVirtualLevel",
            "setTargetHumidity",
            "setHumidityMode",
            "setDisplay",
            "setDryingMode"
          ]
        }
      ],
      "outlets": [],
      "switches": []
    }
  }
}
2024-11-09 10:35:23.927 WARNING (SyncWorker_1) [py.warnings] /usr/local/lib/python3.12/site-packages/urllib3/connectionpool.py:1064: InsecureRequestWarning: Unverified HTTPS request is being made to host '192.168.1.10'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(

2024-11-09 10:36:22.392 ERROR (MainThread) [homeassistant.config_entries] Error unloading entry ***REDACTEDEMAILADDRESS*** for light
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 850, in async_unload
    result = await component.async_unload_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/light/__init__.py", line 680, in async_unload_entry
    return await hass.data[DATA_COMPONENT].async_unload_entry(entry)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in async_unload_entry
    raise ValueError("Config entry was never loaded!")
ValueError: Config entry was never loaded!

@saladm2
Copy link

saladm2 commented Nov 15, 2024

Would love to see this added too

@gaitskell
Copy link

I have the Levoit OasisMist 4.5L humidifier.

It can be controlled in their phone App correctly - reports temperature, humidity. Turns device on and off and selects mode Automatic/manual.

It is not recognized in the HASS VeSync AddOn. Device does not appear. This humidifier appears outside the current listed spec for the AddOn. Could it be added?

@bmos
Copy link

bmos commented Nov 17, 2024

I have the Levoit OasisMist 4.5L humidifier.

It can be controlled in their phone App correctly - reports temperature, humidity. Turns device on and off and selects mode Automatic/manual.

It is not recognized in the HASS VeSync AddOn. Device does not appear. This humidifier appears outside the current listed spec for the AddOn. Could it be added?

That is a different product than the 6000S. You should create a feature request.

@pairot
Copy link

pairot commented Nov 19, 2024

Another request to add Levoit 6000S humidifier

@bmos
Copy link

bmos commented Nov 21, 2024

There is a feature request thread in the proper place (the forum) for this, btw.

https://community.home-assistant.io/t/add-support-for-levoit-superior-6000s-smart-evaporative-humidifier-in-the-vesync-integration/656429

@ncd7
Copy link

ncd7 commented Nov 23, 2024

I tend to think going directly to GitHub is more effective when it isn't regarding core HA features. Asking for specific integration / device supports on the feature request will not do much since most integrations are not owned by the core HA team. They start in the community and get absorbed. And even once absorbed as VeSync, the owners tend to remain community contributors. Whereas for core HA features, they are actually mostly owned by the HA on-staff devs. I might be wrong on this, just my impression.

@cdnninja
Copy link
Contributor

Does the underlying library support this device? If not please create a request in that GitHub.

@bmos
Copy link

bmos commented Nov 25, 2024

I opened an issue requesting support from pyvesync which I think is the library that is used:
webdjoe/pyvesync#256

@x10
Copy link
Author

x10 commented Nov 27, 2024

There is a feature request thread in the proper place (the forum) for this, btw.

https://community.home-assistant.io/t/add-support-for-levoit-superior-6000s-smart-evaporative-humidifier-in-the-vesync-integration/656429

@bmos I have been following in the forums for over a year of ownership of the 6000s and nothing happened so I popped over to GH.

@x10
Copy link
Author

x10 commented Dec 11, 2024

There is a feature request thread in the proper place (the forum) for this, btw.
https://community.home-assistant.io/t/add-support-for-levoit-superior-6000s-smart-evaporative-humidifier-in-the-vesync-integration/656429

@bmos I have been following in the forums for over a year of ownership of the 6000s and nothing happened so I popped over to GH.

Yeah, same, I've been watching the forum for almost a year and just decided to move to GH and make this issue.

There is also a promising PR on the Vesync custom HACS that looks promising and is waiting for review.

micahqcade/custom_vesync#11

🤞 we get something that works soon!

@cdnninja
Copy link
Contributor

#131433 is the first step toward this.

@sirabdude
Copy link

There is a feature request thread in the proper place (the forum) for this, btw.
https://community.home-assistant.io/t/add-support-for-levoit-superior-6000s-smart-evaporative-humidifier-in-the-vesync-integration/656429

@bmos I have been following in the forums for over a year of ownership of the 6000s and nothing happened so I popped over to GH.

Yeah, same, I've been watching the forum for almost a year and just decided to move to GH and make this issue.

There is also a promising PR on the Vesync custom HACS that looks promising and is waiting for review.

micahqcade/custom_vesync#11

🤞 we get something that works soon!

Yep! If you're using HACS you can install from my fork right now and control your 6000s. Been waiting a while for review so unfortunately I don't know when it'll be approved :(

@saladm2
Copy link

saladm2 commented Dec 14, 2024

There is a feature request thread in the proper place (the forum) for this, btw.
https://community.home-assistant.io/t/add-support-for-levoit-superior-6000s-smart-evaporative-humidifier-in-the-vesync-integration/656429

@bmos I have been following in the forums for over a year of ownership of the 6000s and nothing happened so I popped over to GH.

Yeah, same, I've been watching the forum for almost a year and just decided to move to GH and make this issue.
There is also a promising PR on the Vesync custom HACS that looks promising and is waiting for review.
micahqcade/custom_vesync#11
🤞 we get something that works soon!

Yep! If you're using HACS you can install from my fork right now and control your 6000s. Been waiting a while for review so unfortunately I don't know when it'll be approved :(

Hey, your fork w/ 6000s support is working great for me, thanks! I've been using Google assistant inside of HA to turn it on and off depending on my external humidity sensor, but you can't set the mist level with gAssistant. But now I can with your fork - awesome!

@ncd7
Copy link

ncd7 commented Dec 15, 2024

#131433 is the first step toward this.

Did you see the comment from #131433 (review)
Does your fork not face this issue?
The fork is amazing but I'm trying to reduce the number of custom forks I have so hoping your PR can be merged once that issue is resolved.

Happy Holidays and thanks for doing this! <3

@cdnninja
Copy link
Contributor

I'm not sure why the tests fail. I don't know the integration well enough to tell for sure.

@x10
Copy link
Author

x10 commented Dec 24, 2024

#131433 is the first step toward this.

@cdnninja just subscribed to the HA core PR. Thanks for working on this. Looks like a lot going on with the ecosystem and tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants