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

Unable to specify power sensor during configuration #319

Closed
JonathanJorstad opened this issue Dec 10, 2023 · 9 comments
Closed

Unable to specify power sensor during configuration #319

JonathanJorstad opened this issue Dec 10, 2023 · 9 comments
Labels
area:chargertype Issue regarding chargertype bug Something isn't working

Comments

@JonathanJorstad
Copy link

I have tried specifying both the sensor entity ID and the name, but they do not seem to be recognized. I am using a the AMSleser powermeter with MQTT.

During configuration, it is unclear whether the power sensor entity ID or entity name should be specified - or device name? There is no drop down/quick search results as you type in the configuration text box.

Skjermbilde 2023-12-10 120553
Skjermbilde 2023-12-10 120328
Skjermbilde 2023-12-10 120314

Error log:
Loggdetaljer (FEIL)
Logger: homeassistant.config_entries
Source: config_entries.py:402
First occurred: 12:07:42 (1 occurrences)
Last logged: 12:07:42

Error setting up entry sensor.ams_p for peaqev
Traceback (most recent call last):
File "/config/custom_components/peaqev/peaqservice/chargertypes/chargertype_factory.py", line 62, in async_create
ret.is_initialized = await ret.async_setup()
^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/peaqservice/chargertypes/icharger_type.py", line 78, in async_setup
update_current=self.call_update_current,
^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/peaqservice/chargertypes/types/zaptec.py", line 86, in call_update_current
raise NotImplementedError
NotImplementedError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 402, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/init.py", line 28, in async_setup_entry
hub = await HubFactory.async_create(hass, options, DOMAIN)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/peaqservice/hub/hub_factory.py", line 36, in async_create
return await HubFactory.async_setup(hub(hass, options, domain))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/peaqservice/hub/hub_factory.py", line 40, in async_setup
hub.chargertype = await ChargerTypeFactory.async_create(hub.state_machine, hub.options)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/peaqev/peaqservice/chargertypes/chargertype_factory.py", line 77, in async_create
raise Exception
Exception

@JonathanJorstad JonathanJorstad changed the title Unable to specify power sensor during setup Unable to specify power sensor during configuration Dec 10, 2023
@elden1337
Copy link
Owner

seems your powersensor is correctly put (sensor.ams_p as your entry is called is the right way). but it seems as if the zaptec-integration handling is being angry for some reason .is this all you get from peaqev?

@elden1337 elden1337 added bug Something isn't working area:chargertype Issue regarding chargertype labels Dec 10, 2023
@elden1337 elden1337 self-assigned this Dec 10, 2023
@elden1337
Copy link
Owner

i'm going to push a quick hotfix-update for you which is hope should make it better.
please note that zaptec integration has yet to get update-current features since they aren't ready (or ready just now perhaps but i haven't had time to implement them yet)

elden1337 pushed a commit that referenced this issue Dec 10, 2023
elden1337 pushed a commit that referenced this issue Dec 10, 2023
@elden1337
Copy link
Owner

there: https://github.com/elden1337/hass-peaq/releases/tag/3.3.1

hopefully it will work better for you. keeping this ticket open til you say it's ok.

@JonathanJorstad
Copy link
Author

I think it worked, at least I am now seeing the 4 devices in peaqev.

@elden1337
Copy link
Owner

splendid.
As mentioned i don't have updatecurrent for zaptec (perhaps it's ready to implement by now but noones asked for it lately). I'll have a look at it.

Happy charging!

@elden1337
Copy link
Owner

hmm, looks like they've broken the whole integration. but you are able to get peaqev running without any seen issues now? @JonathanJorstad ?

@JonathanJorstad
Copy link
Author

JonathanJorstad commented Dec 10, 2023 via email

@elden1337
Copy link
Owner

Yepp. Got it. The breaking change from them has made it useless. I have to refactor the code. Sorry about that but I couldnt foresee that change on their part.

@elden1337
Copy link
Owner

New feature for it is #320

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:chargertype Issue regarding chargertype bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants