PyPSA / PyPSA/pypsa-usa

Change `get_energy_timeseries` function to pypsa dispatch function

Open
#242 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
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:

  1. (current) get_energy_timeseries(n)
  2. (proposed) StatisticsAccessor(n).dispatch(aggregate_time=False).T
Additional Info

No response

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.