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

Rollease Acmeda integration fails with "UnboundLocalError: cannot access local variable 'hubs' where it is not associated with a value" #124025

Open
davros123 opened this issue Aug 16, 2024 · 6 comments · Fixed by atmurray/ha-acmeda#3

Comments

@davros123
Copy link

The problem

Integration fails to start with this error when trying to add a hub (v1).
image - looking at the logs

I was running the HACS version of this integration however it was not reliably working (only updated after reboot then stopped responding after a day or so) so I decided to uninstall the HACS and move to the "official" supported integration.

When I tried to add the "official" integration, I revieve the above error. After experiencing the above error on my (fully configured) HA, I thought it might be my config...so I wiped the ssd and installed a completely fresh install with no other integrations or config. and got the same error.

(same error from my orginall (fully setup) HA) image

What version of Home Assistant Core has the issue?

core-2024.8.1

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

Rollease Acmeda Automate

Link to integration documentation on our website

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

Diagnostics information

Here is the error
acmedaerror.txt

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 @atmurray, mind taking a look at this issue as it has been labeled with an integration (acmeda) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of acmeda 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 acmeda 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)


acmeda documentation
acmeda source
(message by IssueLinks)

@AdamCavaliere
Copy link

I got this to work by adding it as a custom integration through HACS: https://github.com/atmurray/ha-acmeda

@parkglen
Copy link

I got this to work by adding it as a custom integration through HACS: https://github.com/atmurray/ha-acmeda

Thanks @AdamCavaliere - That worked

atmurray added a commit to atmurray/ha-acmeda that referenced this issue Aug 31, 2024
Resolve home-assistant/core#124025 due to bad refactoring of code.
@atmurray
Copy link
Contributor

Hi all, I've got a pull request open that fixes this issue:
#124960
Until that gets accepted/merged/released, this is the same as the v0.4.6 release of the custom integration you can add using HACS:
https://github.com/atmurray/ha-acmeda

@davros123
Copy link
Author

Thanks I updated the HACS version and it resolved this issue.
It "may" have also resolved an issue where the blinds stopped responding after a day or so...it's been 24 hours and still working 🤞

Thanks @atmurray for your continued support of this integration

@papaavi
Copy link

papaavi commented Sep 10, 2024

Hey @atmurray this solved for me too. I noticed when I did this solution last time for a different issue via HACs, the HASS version of the integration duplicated my blinds so I had 2x of the same (I deleted the integration so that's how I got in this current 500 error mess in the first place). Thank you so much for supporting this integration, it's the whole reason I got into HASS in the first place. It's brilliant.

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

Successfully merging a pull request may close this issue.

5 participants