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

Length_factor multiplied twice #1412

Open
lindnemi opened this issue Nov 7, 2024 · 0 comments
Open

Length_factor multiplied twice #1412

lindnemi opened this issue Nov 7, 2024 · 0 comments
Assignees
Labels

Comments

@lindnemi
Copy link
Contributor

lindnemi commented Nov 7, 2024

I just had a thorough look with @p-glaum at the length_factor. As far as we understand, in the clustered network, the length of all lines is set to the haversine distance between the buses multiplied by length_factor.

https://github.com/PyPSA/PyPSA/blob/068ca0b3397194390493ccc140ad0c79a2ea7310/pypsa/clustering/spatial.py#L397-L400

However, the capital costs in add_electricity consider the length factor a second time:

n.lines["capital_cost"] = (
n.lines["length"]
* line_length_factor
* costs.at["HVAC overhead", "capital_cost"]
)

Thus we might be overestimating the capital_cost of lines, with the default setting by 25%. The same probably also applies to links.

@lindnemi lindnemi added the bug label Nov 7, 2024
@lindnemi lindnemi changed the title Length_factor multiplied twiche Length_factor multiplied twice Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants