Skip to content

use grpc-requests#410

Merged
peterdudfield merged 2 commits intomainfrom
use-grpc-request
Mar 30, 2026
Merged

use grpc-requests#410
peterdudfield merged 2 commits intomainfrom
use-grpc-request

Conversation

@peterdudfield
Copy link
Copy Markdown
Contributor

@peterdudfield peterdudfield commented Mar 30, 2026 •
edited
Loading

Pull Request

Description

  • change to grpc_requests for big forecast data pull
    This speeds things up.
    For loading pvnet_v2 to load all data, it used to take ~5 seconds, now it takes ~1 seconds.

How Has This Been Tested?

  • CI tests
  • tried it locally
  • deployed to dev, and did speed tests

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

@peterdudfield peterdudfield marked this pull request as draft March 30, 2026 17:32
@peterdudfield peterdudfield marked this pull request as ready for review March 30, 2026 19:52
@peterdudfield peterdudfield merged commit abf40a4 into main Mar 30, 2026
5 checks passed
@peterdudfield peterdudfield deleted the use-grpc-request branch March 30, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@peterdudfield