-
Notifications
You must be signed in to change notification settings - Fork 20
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
Updating all references to deprecation warnings for 2025.1 #50
base: master
Are you sure you want to change the base?
Updating all references to deprecation warnings for 2025.1 #50
Conversation
Also can't figure out how to use Dev Containers. Would love it if someone could show me the ropes. |
Thanks for your contribution! Good to see some action. |
Well, first time contributing to anything from Home Assistant as I'm rather new, and my first time working in python, even though I've been coding for years. As the repository has been required for so long hopefully someone is willing to take the time to look at my work and maybe , they actually deem it worthy |
Is there a way I could help test? What’s needed to get this committed? |
@GitittomeNow I don't know. Having heard nothing from @dotKrad I don't know how if the owner of the repo isn't active. I'm gonna install on my local HA and test it. I also formed and started a rewrite using a template from one of the HA core contributors, but it's my first python project so much more of a learning curve than just editing a couple files. |
My biggest issue currently is i have net metering (cause I have solar) and am getting issue with the negative values returned. I think i need to add another sensor to sum something, /shrug |
I’d be happy to help in someway! I’m an IT engineer but have not done coding yet. I too have net metering along with batteries. For me, everything shows properly in an add-on called “Power Flow Card Plus” but not the Home Assistant native energy dashboard. Also, the current version gives all kinds of errors each day, which is annoying. |
I tested this changes and it seems to get rid of the deprecation warnings. I also updated https://github.com/dotKrad/hass-fpl/blob/master/custom_components/fpl/sensor_HourlyUsageSensor.py in the same way. Also there was a warning about async_add_job() in init.py ...
I changed that to use async_create_task() instead and the warning went away. I still have some errors in the logs though... `2024-05-17 23:45:04.570 WARNING (MainThread) [homeassistant.components.backup] fpl does not implement required functions for the backup platform 2024-05-18 00:17:55.129 ERROR (MainThread) [custom_components.fpl] ` I still don't think my energy dashboard is populating correctly though. Which sensor should I be using in the energy dashboard configuration? |
I recent installed this and had a ton of warnings. First time in Python, please check my work thoroughly.