dont loop, use snapshot method#191

Open

peterdudfield wants to merge 2 commits intomainfrom

improve-forecast-all-route

Contributor

peterdudfield commented Feb 19, 2026 •

edited

Loading

Pull Request

Description

change foreacst/all route to use get_forecast_at_timestamp
It takes 10 seconds, but I understand its being speed up

How Has This Been Tested?

CI tests
tested it locally
test on dev

Checklist:

My code follows OCF's coding style guidelines
I have performed a self-review of my own code
I have made corresponding changes to the documentation
I have added tests that prove my fix is effective or that my feature works
I have checked my code and corrected any misspellings


          dont loop, use snapshot method

1c2a951

peterdudfield mentioned this pull request

bug: forecast/all one timestamp in the future #190

Closed

fix

1264ee3

peterdudfield requested a review from devsjc

February 19, 2026 17:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet