Skip to content

feat(power): expose opt-in MW debiasing - #87

Merged
mroberto166 merged 2 commits into
mainfrom
feat/power-debias
Aug 14, 2026
Merged

feat(power): expose opt-in MW debiasing#87
mroberto166 merged 2 commits into
mainfrom
feat/power-debias

Conversation

@mroberto166

Copy link
Copy Markdown
Contributor

Summary

  • add debias=False to PowerForecast.get_data() and get_day_ahead_timeseries()
  • add debias=False to EnergyMarket.compare_runs_mw()
  • omit the field from outgoing requests unless explicitly enabled, preserving existing raw outputs

Test plan

  • uv run just check-commit
  • 219 unit tests passed
  • mypy, Ruff, formatting, and pre-commit checks passed
  • functional CI against the live API

Release dependency

Market-aggregate support depends on jua-core #6177 being merged and deployed to Query Engine. Do not merge or release this SDK PR before the live endpoint accepts debias=true.

Made with Cursor

Allow Power Forecast and market-aggregate users to request leakage-safe MW debiasing while retaining byte-compatible raw request bodies by default.

Verification: - jua-core PowerForecastQuery.debias and ForecastQuery.mw_walkforward_debias default false
  - uv run just check-commit passes
Co-authored-by: Cursor <cursoragent@cursor.com>
@mroberto166
mroberto166 requested a review from a team as a code owner August 12, 2026 16:11
cursor[bot]
cursor Bot previously approved these changes Aug 12, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale comment

Approved based on a successful Cursor Security Agent check with no findings requiring human review; Cursor Bugbot was not present on this PR. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@philipp-seitz-jua
philipp-seitz-jua self-requested a review August 13, 2026 08:45
niasie
niasie previously approved these changes Aug 13, 2026
Drop leakage-safe wording and explain that wind uses eight weeks and
solar four weeks to estimate the bias, which is then subtracted before
the window moves ahead.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved based on a successful Cursor Security Agent check with no findings requiring human review; Cursor Bugbot was not present on this PR. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@mroberto166
mroberto166 merged commit fbf7140 into main Aug 14, 2026
48 checks passed
@mroberto166
mroberto166 deleted the feat/power-debias branch August 14, 2026 08:00
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.

4 participants