Skip to content

Support requesting specific resolution#463

Merged
slovdahl merged 6 commits intomainfrom
support-requesting-specific-resolution
Oct 4, 2025
Merged

Support requesting specific resolution#463
slovdahl merged 6 commits intomainfrom
support-requesting-specific-resolution

Conversation

@slovdahl
Copy link
Owner

@slovdahl slovdahl commented Oct 4, 2025

No description provided.

@slovdahl slovdahl added the new-feature New features or options. label Oct 4, 2025
@slovdahl slovdahl requested a review from Copilot October 4, 2025 16:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for requesting specific price resolution intervals when fetching energy prices. The main change allows users to specify a 15-minute resolution interval in addition to the default 60-minute resolution.

  • Added a resolution parameter to the energy_prices method with support for 15 and 60-minute intervals
  • Added validation to reject unsupported resolution values
  • Added comprehensive test coverage for the new 15-minute resolution functionality

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
tests/test_models.py Added tests for 15-minute resolution and unsupported resolution validation
tests/fixtures/energy-15-min.json Added fixture data for 15-minute resolution testing
spothinta_api/spothinta.py Modified energy_prices method to accept resolution parameter with validation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@slovdahl slovdahl merged commit 6632df9 into main Oct 4, 2025
13 checks passed
@slovdahl slovdahl deleted the support-requesting-specific-resolution branch October 4, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature New features or options.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants