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
add implementation of giving it live pv data #47
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #47 +/- ##
=========================================
+ Coverage 0 79.87% +79.87%
=========================================
Files 0 11 +11
Lines 0 333 +333
=========================================
+ Hits 0 266 +266
- Misses 0 67 +67 ☔ View full report in Codecov by Sentry. |
Hi @peterdudfield ,what does this pr do? Is this similar to #106 that was built on #66 |
Can this be merged? |
Yea, I let this sit there for a bit. I think what needs to happen is
Do you fancy doing that @aryanbhosale |
Its realted to #66, but this mean users can enter their live PV data into the model. And yea, this could be done using the enphase API, or other methods. Wanted to also run the evaluation to show how much better the model is by using this data |
Oh that's great, are there any changes needed here ? |
Totally up to you. You could build off this branch, or start a fresh, totally up to you. |
Pull Request
Description
TODO - run this with evaluation, perhaps with 1 hour of live pv data
Fixes #36
How Has This Been Tested?
Added an example and TODO run evaluation
Checklist: