Skip to content
This repository was archived by the owner on Mar 27, 2025. It is now read-only.

Bugfixing Transport Sector from Aladin#127

Merged
lindnemi merged 1 commit intomainfrom
aladin_bug_fix
Jun 25, 2024
Merged

Bugfixing Transport Sector from Aladin#127
lindnemi merged 1 commit intomainfrom
aladin_bug_fix

Conversation

@toniseibold
Copy link
Copy Markdown
Contributor

Factor for Vehicle2Grid link, BEV_charger and Demand Side Management was not applied due to non matching indices.

@toniseibold toniseibold marked this pull request as ready for review June 25, 2024 13:19
@toniseibold toniseibold requested a review from lindnemi June 25, 2024 13:19
@lindnemi
Copy link
Copy Markdown
Collaborator

What about n.stores.carrier == "Li ion"?

There are too many confusingly labeled battery carriers in the model:

  • battery -> i think for utilitiy scale batteries
  • home battery -> home batteries
  • Li ion -> so far i assumed these are the batteries in the BEVs and should be scaled with the factor from aladin
  • battery storage -> supposedly also for demand side management of BEVs, i saw it in the code, but it does not turn up in the postnetworks, no idea why it is missing

@lindnemi
Copy link
Copy Markdown
Collaborator

I talked to Lisa about this. It's a bug in PyPSA that has been fixed about a month ago. Apparently the optimisation changed the carrier of stores to match the carrier of the bus to which it connects.

So this PR is probably correct, but i have to update my PyPSA version. Addtionally i will prepare a PR for pypsa-eur which renames the carriers

@lindnemi lindnemi merged commit b02295c into main Jun 25, 2024
@toniseibold toniseibold deleted the aladin_bug_fix branch October 25, 2024 11:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants