We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
For pv sites we currently aren't testing against our actual models and instead are using the dummy model https://github.com/openclimatefix/india-forecast-app/blob/main/tests/test_app.py#L81 which generates fake predictions, I imagine this was done previously when pv models weren't quite ready but perhaps now it feels like the right time to remove this to improve our model test coverage.
A wider question around whether the dummy model should be kept in, could be useful for other mocking/testing.
The text was updated successfully, but these errors were encountered:
Could you assign this issue to me? I would love to contribute :)
Sorry, something went wrong.
No branches or pull requests
For pv sites we currently aren't testing against our actual models and instead are using the dummy model https://github.com/openclimatefix/india-forecast-app/blob/main/tests/test_app.py#L81 which generates fake predictions, I imagine this was done previously when pv models weren't quite ready but perhaps now it feels like the right time to remove this to improve our model test coverage.
A wider question around whether the dummy model should be kept in, could be useful for other mocking/testing.
The text was updated successfully, but these errors were encountered: