Skip to content

Gsp forecast compact false#222

Merged
peterdudfield merged 13 commits intomainfrom
gsp-forecast-compact-false
Mar 24, 2026
Merged

Gsp forecast compact false#222
peterdudfield merged 13 commits intomainfrom
gsp-forecast-compact-false

Conversation

@peterdudfield
Copy link
Copy Markdown
Contributor

@peterdudfield peterdudfield commented Mar 23, 2026

Pull Request

Description

Adds a compact=false option to forecast/all
This matches previous API

This needs up an upgrade in the dataplatform to pass creation and initi times in the snapshot route. We could wait until that is done first (openclimatefix/data-platform#136)

Fixes #

How Has This Been Tested?

  • Ci Tests (added extra tests)

https://github.com/openclimatefix/client-private/issues/267

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 23, 2026 18:08
@peterdudfield peterdudfield marked this pull request as ready for review March 23, 2026 18:40
@peterdudfield peterdudfield requested a review from devsjc March 23, 2026 19:43
@peterdudfield peterdudfield marked this pull request as draft March 23, 2026 19:44
@peterdudfield peterdudfield marked this pull request as ready for review March 23, 2026 20:40
…cast-compact-false

# Conflicts:
#	src/quartz_api/tests/integration/uk_national/test_app.py
…cast-compact-false

# Conflicts:
#	src/quartz_api/internal/service/uk_national/gsp_router.py
@peterdudfield peterdudfield merged commit ce3ebbc into main Mar 24, 2026
8 checks passed
@peterdudfield peterdudfield deleted the gsp-forecast-compact-false branch March 24, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants