Nuevo indicador cuartohorario para el Precio Medio de Desvíos#66
Conversation
davidmunoznovoa
changed the title
There was a problem hiding this comment.
Pull request overview
This PR adds a new quarter-hourly variant indicator for measured deviations in the free market, along with its corresponding test. The new indicator mhpMeasuredDeviationsFreeQh uses the same API endpoint as the hourly variant but retrieves data at 15-minute granularity.
Changes:
- Added
mhpMeasuredDeviationsFreeQhclass to support quarter-hourly time granularity for indicator 799 - Added test case validating the new indicator's behavior and expected API response
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| esios/indicators.py | Added new mhpMeasuredDeviationsFreeQh indicator class with fifteen_minutes time truncation, sharing path with existing hourly variant |
| spec/indicators_spec.py | Added test case in new "Medium Quarter-Hourly Price Components" context to verify the new indicator |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| class mhpMeasuredDeviationsFreeQh(Indicator): | ||
| path = 'indicators/799' | ||
| time_trunc = 'fifteen_minutes' |
There was a problem hiding this comment.
Missing blank line before class definition. The codebase consistently uses two blank lines between class definitions (see lines 186-202 for examples). Add a blank line before this class definition to maintain consistency with the established coding style.
|
|
||
| class mhpMeasuredDeviationsFreeQh(Indicator): | ||
| path = 'indicators/799' | ||
| time_trunc = 'fifteen_minutes' |
There was a problem hiding this comment.
Missing blank line after class definition. The codebase consistently uses two blank lines between class definitions (see surrounding code at lines 190-202). Add a blank line after this class definition to maintain consistency with the established coding style.
| ) | ||
| with context('Medium Quarter-Hourly Price Components'): | ||
| with it('Returns mhpMeasuredDeviationsFreeQh'): | ||
| #799 |
There was a problem hiding this comment.
Missing space after comment symbol. The codebase consistently uses a space after the hash symbol in comments (see lines 246, 261, 580, 594, 608, etc.). Change this to include a space after the hash.
davidmunoznovoa
deleted the
84512_new_mhpMeasuredDeviationsFreeQh_indicator
branch
Objetivos
Añadir los siguientes indicadores cuartohorarios a la librería:
799:Precio medio horario componente desvíos medidos contratación libreImplementar tests para los indicadores.
Relacionado