-
Notifications
You must be signed in to change notification settings - Fork 13
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
Include interest area in AC clustering #662
base: dev
Are you sure you want to change the base?
Conversation
Hi @ulfmueller this PR is ready for review. According to our agreement with @birgits and David, now it is possible to use also the eHV clustering while keeping the buses inside the interest area. It means that there are no 110kV lines or buses outside the provided area, therefore it is very important to define carefully the shapes since it could lead to 110kV isolated buses. |
As examples of well-defined areas, we can take the two shapefiles created and shared by @ulfmueller (Fokusregion.zip), where the network topology is taken into account and no stubs can be found. Using the mentioned areas, the clustering process runs as expected. |
@CarlosEpia I added the option, to plot the nodes inside the interest area differently from the other nodes. You can remove it if it's not helpful or anything. I just added it as a suggestion as I implemented it for myself anyway. |
In case components are replaced by an extension scenario these are always dropped. Thus, only the argument scn_extension is needed
Features/extendable lines from nep
area_network = network_based_on_buses(etrago.network, buses_area) | ||
|
||
elif apply_on == "market_model": | ||
elec_network_buses = etrago.network_tsa.buses[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CarlosEpia or @ClaraBuettner, I discovered that n_clusters is not assigned in this elif. Not sure if this was done in this branch or not, so maybe it's not an issue that should be fixed here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is defined in cluster.electrical.py on dev
. In that function, n_clusters is defined for all settings and the clustering works fine. So I guess it is related to this branch. Could you have a look at it @CarlosEpia?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When scn_extension is selected, some HV buses are connected to more than one eHV bus. This leads to duplicated indices in the path matrix. To avoid this, each HV bus is only considered once using transformers.bus1.unique().
This is needed when some e-Mobility buses are not clustered. In some cases, the driving load is not high enough to match the e_min_pu limits of the ev stores.
Add option for negative load shedding at e-Mobility buses
Solves the AC and associated sectors of #661. The areas can be supplied by path to shapefiles or list of nuts names e.g., ["Cuxhaven", "Bremerhaven", "Wesermarsch", "Osterholz", "Bremen"].