chore: remove old data config#78
Conversation
|
Hi @AnujNayak108, thanks for putting this up but I can see that the CI runs have failed, and since there hasn't been any work done on this PR in a while I am going to close this. Let me know if you want to pick this back up! |
|
Hi @AUdaltsova I’d like to continue working on this PR. What I noticed the tests were failing because some of the data_config.yaml files could not be downloaded from Hugging Face during CI, and I wasn’t sure whether this was a network issue, a missing mock, or something expected in this repo. If you could point me to the relevant failing step or confirm the expected behaviour, I’d be happy to fix it properly and reopen the PR. |
|
Hi @AnujNayak108, thanks for getting back to me & for being ready to work on the issue! I've reopened the PR now. Very odd that the tests are not passing, I've merged a couple of PRs recently and had no CI issues. Not sure what's wrong here! |
Pull Request
Description
This PR removes the unused live_delay_minutes field from site_forecast_app/models/pvnet/utils.py.
After searching through:
This repo → only found in utils.py, where it was immediately removed from satellite_config.
Config generation repo → not present.
HuggingFace configs → not present in the satellite section or elsewhere.
Therefore, this field is no longer needed and can safely be removed.
Fixes #74
Checklist: