Bugfixing Transport Sector from Aladin

PyPSA / pypsa-ariadne

MIT License

6 stars 4 forks source link

Bugfixing Transport Sector from Aladin #127

Closed toniseibold closed 1 week ago

toniseibold commented 1 week ago

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

lindnemi commented 1 week ago

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 commented 1 week ago

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