Change `get_energy_timeseries` function to pypsa dispatch function
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 148
- Forks
- 55
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 18
Description
Feature Request
We should change our get_energy_timeseries(n) function to the pypsa.statistics.StatisticsAccessor.dispatch() method. The advantage of this is that we can simplify our code and take advantage of pypsa testing their functions. Note, we need to double check we care assuming same logic for link power dispatch
Suggested Solution
I believe these two are equivalent:
- (current)
get_energy_timeseries(n) - (proposed)
StatisticsAccessor(n).dispatch(aggregate_time=False).T
Additional Info
No response
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the current get_energy_timeseries(n) implementation and compare its behavior with StatisticsAccessor(n).dispatch(aggregate_time=False).T. Check specifically whether link power dispatch follows the same logic, then run the existing tests or add coverage for the comparison; done means the replacement preserves the required timeseries behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100