Skip to content

Catches small but important bugs with branch partitioning. #6

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.

Sign up for GitHub

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

Merged
dpinney-nreca merged 1 commit into from
Nov 6, 2025
Merged

Catches small but important bugs with branch partitioning. #6

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions microgridup_gen_mgs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,9 +190,8 @@ def nx_group_branch(G, i_branch=0, omd={}):
bbl = nx_get_branches(edges_in_order)
bbl_indices_to_delete = set() # Add a check to ensure branch algo does not consider item level for branching when user created circuit with wizard.
for idx, bbl_item in enumerate(bbl):
bus = bbl_item[0]
if bus.endswith('_end'):
feeder_name = bus[0:-4]
if bbl_item.endswith('_end'):
feeder_name = bbl_item[0:-4]
ob_type = get_object_type_from_omd(feeder_name, omd)
if ob_type == 'line':
bbl_indices_to_delete.add(idx)
Expand Down Expand Up @@ -583,8 +582,9 @@ def form_microgrids(G, MG_GROUPS, omd, switch_dict=None, gen_bus_dict=None, mg_n
print(f'Selected partitioning method produced invalid results. Please change partitioning parameter(s).')
raise SwitchNotFoundError(f'Selected partitioning method produced invalid results. Please change partitioning parameter(s).')
if not mg.get('loads'):
print(f'Partitioning produced microgrid "{mg_name}" with no loads. Please ensure each MG has at least one load.')
raise ValueError(f'Partitioning produced microgrid "{mg_name}" with no loads. Please ensure each MG has at least one load.')
msg = f'Partitioning method produced microgrid "{mg_name}" with no loads. Please choose a different partitioning method or ensure each microgrid has at least one load.'
print(msg)
raise ValueError(msg)
return MICROGRIDS

def _tests():
Expand Down
2 changes: 1 addition & 1 deletion microgridup_gui.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -779,7 +779,7 @@ def _get_microgrids(critical_loads, partition_method, quantity, dss_path, partit
mg_groups = form_mg_groups(G, critical_loads, 'lukes', json.loads(partition_params_json))
microgrids = form_microgrids(G, mg_groups, omd)
elif partition_method == 'branch':
mg_groups = form_mg_groups(G, critical_loads, 'branch')
mg_groups = form_mg_groups(G, critical_loads, 'branch', algo_params={'omd':omd})
microgrids = form_microgrids(G, mg_groups, omd)
elif partition_method == 'bottomUp':
mg_groups = form_mg_groups(G, critical_loads, 'bottomUp', algo_params={'num_mgs':quantity, 'omd':omd, 'cannot_be_mg':['regcontrol']})
Expand Down