openego / eTraGo

Optimization of flexibility options for transmission grids based on PyPSA
GNU Affero General Public License v3.0
30 stars 13 forks source link

Fixes/#660 fix ehv clustering #676

Closed CarlosEpia closed 8 months ago

CarlosEpia commented 9 months ago

Deals with #660

CarlosEpia commented 9 months ago

Finally all the checks passed. Ready for review.

CarlosEpia commented 9 months ago

Comments already considered. Ready for a new review.

ClaraBuettner commented 8 months ago

I get this warning when I run the kmedoids-dijkstra clustering after the ehv clustering:

INFO:etrago.cluster.electrical:Network clustered to EHV-grid
INFO:etrago.cluster.electrical:Start k-medoids Dijkstra Clustering
INFO:etrago.cluster.electrical:There are 3 buses that have no direct
            connection to the electric network: ['77783', '78329', '78721']

Do you have the same warning? It looks as if it should not be there...

CarlosEpia commented 8 months ago

I get this warning when I run the kmedoids-dijkstra clustering after the ehv clustering:

INFO:etrago.cluster.electrical:Network clustered to EHV-grid
INFO:etrago.cluster.electrical:Start k-medoids Dijkstra Clustering
INFO:etrago.cluster.electrical:There are 3 buses that have no direct
            connection to the electric network: ['77783', '78329', '78721']

Do you have the same warning? It looks as if it should not be there...

There were 3 not electric buses being clustered based on eHV foreign buses. When the Kmedoid-Dijkstra clustering was used excluding the foreign countries, the mentioned buses were isolated. The commit 92aeaf1 includes changes to find the closest eHV bus in Germany for those cases.