Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add TSAM script and deps #87

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Conversation

henhuy
Copy link
Collaborator

@henhuy henhuy commented May 7, 2024

TODOS:

  • make linter happy
  • fix too many sequences and strange file prefixes

@srhbrnds
Copy link
Collaborator

srhbrnds commented May 8, 2024

The script time_series_aggregation.py is not yet generic enough to automatically aggregate all possible sequence data with tsam and return a new oemof-tabular datapackage. This is due to the partly different naming of the sequence csv-files and the column names of the sequence data. I will try to fix this. @henhuy could you send me the oemof-tabular data package with which you tested this?

@srhbrnds
Copy link
Collaborator

I ran into some problems with pre-commit hooks, but i'm still on it to fix this.

@srhbrnds srhbrnds assigned srhbrnds and henhuy and unassigned srhbrnds May 22, 2024
@srhbrnds srhbrnds marked this pull request as ready for review November 12, 2024 19:21
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