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

Fix attribute errors #38

Closed
wants to merge 4 commits into from
Closed

Fix attribute errors #38

wants to merge 4 commits into from

Conversation

drc38
Copy link
Collaborator

@drc38 drc38 commented Jun 9, 2024

@hardbyte, something appears to have changed meaning the dot notation doesn't work for dicts in recent versions of HA. Fixes #36

@hardbyte
Copy link
Owner

I don't think this fix should be required, perhaps it was with an older version of python-evnex?

I just tried with the latest homeassistant docker image and main worked for me.

@drc38
Copy link
Collaborator Author

drc38 commented Jun 23, 2024

I don't think this fix should be required, perhaps it was with an older version of python-evnex?

I just tried with the latest homeassistant docker image and main worked for me.

I can't use evnex >=0.4 due to a pip dependency conflict. My HA pydantic is still on 1.10.15 in docker HA 2024.6.2 and I'm getting still getting the attribute error. Is it likely the two are related?

@drc38
Copy link
Collaborator Author

drc38 commented Jun 25, 2024

@hardbyte, looks like it is pydantic version related as forcing pydantic version to be >=2 gets rid of attribute error. Only problem is a number of other integrations are not yet supporting V2...

@hardbyte
Copy link
Owner

Yeah its not an ideal situation at all. If you like you could branch off the last working 0.4.x version and create a new release that targets the older pydantic.

@drc38
Copy link
Collaborator Author

drc38 commented Jun 27, 2024

Thanks, my HA install is now sorted. Hopefully the HA devs migrate to v2 soon.

@sydney2711
Copy link

Yes the charger is back online BUT I no longer can control the maximum current. I use this to charge the battery at the lowest possible amps.
Therefore this solution isn't for me.
I am going back to 2024.5

@zlodag
Copy link

zlodag commented Dec 16, 2024

Thanks, my HA install is now sorted. Hopefully the HA devs migrate to v2 soon.

@sydney2711 @drc38 Interim fix: #36 (comment)

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

Successfully merging this pull request may close these issues.

Evnex charger not available in 2024.6
4 participants