-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Generation data being collected when it falls below 0kw #197
Comments
@peterdudfield @ADIMANV I’d like to work on this issue as well. Since openclimatefix/ruvnl-consumer-app#20 has already been merged, I believe I can complete this quickly. |
@peterdudfield @ADIMANV I have tried setting up the project locally, but the password is always wrong, first i encountered an error below
on changing the password to an int like 1234 the app runs, but I can't login even with correct password Later just for testing I remove the check in auth.py and got the page below can you also tell how to setup a local db for testing or do we have an db that we can use for testing and development |
@MAYANK12SHARMA could advive how you can run it locally? We should add this to the readme. You have to spin up a local database based on |
I've already begun working on it, but it will take some time as the dashboard requires a fake database and a complete schema. I'm in the process of converting them into a .sql file that includes both the schema and the fake data. So far, I've completed about 40% of the work and will try to upload it within a week. |
@MAYANK12SHARMA any updates ? |
Describe the bug
The data being collected from the SLDC website provided some faulty data. The data contained negative generation, which caused the error calculations for September to be a very high negative percentage. A fix in the backend should be implemented to ignore numbers that are below 0 kW so that our calculations are intact.
This issue is similar to : openclimatefix/ruvnl-consumer-app#20
Expected behavior
The generation data that is seen above where they cross the 0 kW threshold should be completely ignored.
The text was updated successfully, but these errors were encountered: