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.

Sign up for GitHub

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

Jump to bottom

Discrepancy in CTS Demand Assignment to Zensus Cells without CTS Buildings #1180

Open
khelfen opened this issue Feb 3, 2025 · 2 comments
Open

Discrepancy in CTS Demand Assignment to Zensus Cells without CTS Buildings #1180

khelfen opened this issue Feb 3, 2025 · 2 comments
Assignees
@khelfen
Labels
🐛 bug Something isn't working

Comments

@khelfen
Copy link
Contributor

khelfen commented Feb 3, 2025

See openego#362
@khelfen khelfen added the 🐛 bug Something isn't working label Feb 3, 2025
@khelfen khelfen self-assigned this Feb 3, 2025
@khelfen
Copy link
Contributor Author

khelfen commented Feb 3, 2025

At the moment the scenario name is hard coded in the CTS building determination, which leads to incorrect results in the determination if the scenario status2019 is not part of the pipeline.

E.g.

eGon-data/src/egon/data/datasets/electricity_demand_timeseries/cts_buildings.py

Line 322 in 5cd0882

EgonDemandRegioZensusElectricity.scenario == "status2019",

khelfen added a commit that referenced this issue Feb 3, 2025
@khelfen
Merge pull request #1181 from openego/fix/#1180-missing-cts-buildings
5a59dee 
Fix/#1180 missing cts buildings
@khelfen
Copy link
Contributor Author

khelfen commented Feb 3, 2025

#1181 needs to be tested before this issue can be closed

@khelfen khelfen mentioned this issue Feb 3, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@khelfen khelfen

Labels
🐛 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@khelfen