luxtronik icon indicating copy to clipboard operation
luxtronik copied to clipboard

Change the device_info and similar fields to instance variables.

Open Guzz-T opened this issue 1 year ago • 4 comments

Currently, when multiple heat pumps are generated, the data is overwritten in the class variable device_infos. To separate the data of individual heat pumps, there is now a device_infos for each instance.

Fixes #292.

Guzz-T avatar Nov 20 '24 20:11 Guzz-T

@BenPru: Are you satisfied with the changes?

Guzz-T avatar Dec 06 '24 18:12 Guzz-T

@BenPru: Are you satisfied with the changes?

Can anyone other Test this?

BenPru avatar Jan 22 '25 10:01 BenPru

I have this pull request currently in my set up implemented. But I don't have multiple devices in use. What I CAN confirm is: It doe's NOT crash my Integartion.

stefanovich1 avatar Feb 07 '25 12:02 stefanovich1

Can someone merge the branch?

Guzz-T avatar Mar 13 '25 20:03 Guzz-T

Hi @Guzz-T, Thanks for your contribution. I recently joined to help BenPru. I'll do a review and some tests. Feel free to give me a nudge if there is no progress. Let's see if we can get this into the next release.

AJediIAm avatar Jul 06 '25 22:07 AJediIAm