Skip to content

(fix) Set Frequency to None for ERCOT AS Demand Curves DAM and SCED#761

Merged
WillKoehrsen merged 1 commit intomainfrom
set-frequency-to-none
Dec 22, 2025
Merged

(fix) Set Frequency to None for ERCOT AS Demand Curves DAM and SCED#761
WillKoehrsen merged 1 commit intomainfrom
set-frequency-to-none

Conversation

@WillKoehrsen
Copy link
Collaborator

Summary

  • Sets frequency=None in the @support_date_range decorator for Ercot().get_as_demand_curves_dam_and_sced()
  • Since the function is directly filtering for documents by date and end, we don't need a frequency to iterate over days and the "DAY_START" frequency can result in fetching the same file multiple times
  • Make sure the tests still pass: VCR_RECORD_MODE=all uv run pytest gridstatus/tests/source_specific/test_ercot.py -k "as_demand_curves_dam_and_sced"

Details

@WillKoehrsen WillKoehrsen requested a review from a team December 19, 2025 19:47
@WillKoehrsen WillKoehrsen changed the title (fix) Set Frequeny to None for ERCOT AS Demand Curves DAM and SCED (fix) Set Frequency to None for ERCOT AS Demand Curves DAM and SCED Dec 22, 2025
@WillKoehrsen WillKoehrsen merged commit fce007f into main Dec 22, 2025
12 of 18 checks passed
@WillKoehrsen WillKoehrsen deleted the set-frequency-to-none branch December 22, 2025 15:51
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