Skip to content

Commit cf1fe69

Browse files
authored
Add missing transformer tap to the networkx graph (#427)
1 parent ded01e6 commit cf1fe69

File tree

5 files changed

+8
-0
lines changed

5 files changed

+8
-0
lines changed

doc/Changelog.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ og:description: See what's new in the latest release of Roseau Load Flow !
2121

2222
## Unreleased
2323

24+
- {gh-pr}`427` Add missing transformer tap to the edge data in the graph generated by `ElectricalNetwork.to_graph`.
25+
2426
- {gh-pr}`426` Add `rlf.plotting.plot_results_interactive_map` for plotting a network with load flow results on an
2527
interactive map. Buses are colored according to their voltage levels and lines are colored according to their loading
2628
levels.

roseau/load_flow/network.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -416,6 +416,7 @@ def to_graph(self, *, respect_switches: bool = True) -> "MultiGraph":
416416
parameters_id=transformer.parameters.id,
417417
max_loading=transformer._max_loading,
418418
sn=transformer.parameters._sn,
419+
tap=transformer._tap,
419420
geom=geom_mapping(transformer.geometry),
420421
)
421422
for switch in self.switches.values():

roseau/load_flow/tests/test_electrical_network.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2093,6 +2093,7 @@ def test_to_graph(all_element_network: ElectricalNetwork):
20932093
"parameters_id": transformer.parameters.id,
20942094
"max_loading": max_loading,
20952095
"sn": transformer.sn.magnitude,
2096+
"tap": transformer.tap,
20962097
"geom": transformer.geometry.__geo_interface__ if transformer.geometry is not None else None,
20972098
}
20982099

roseau/load_flow_single/network.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -284,6 +284,7 @@ def to_graph(self, *, respect_switches: bool = True) -> "MultiGraph":
284284
id=line.id,
285285
type="line",
286286
parameters_id=line.parameters.id,
287+
length=line._length,
287288
max_loading=line._max_loading,
288289
ampacity=line.parameters._ampacity,
289290
geom=geom_mapping(line.geometry),
@@ -297,6 +298,7 @@ def to_graph(self, *, respect_switches: bool = True) -> "MultiGraph":
297298
parameters_id=transformer.parameters.id,
298299
max_loading=transformer._max_loading,
299300
sn=transformer.parameters._sn,
301+
tap=transformer._tap,
300302
geom=geom_mapping(transformer.geometry),
301303
)
302304
for switch in self.switches.values():

roseau/load_flow_single/tests/test_electrical_network.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1140,6 +1140,7 @@ def test_to_graph(small_network: ElectricalNetwork):
11401140
"parameters_id": line.parameters.id,
11411141
"max_loading": line.max_loading.m,
11421142
"ampacity": ampacity,
1143+
"length": line.length.m,
11431144
"geom": line.geometry.__geo_interface__ if line.geometry is not None else None,
11441145
}
11451146

@@ -1151,6 +1152,7 @@ def test_to_graph(small_network: ElectricalNetwork):
11511152
"parameters_id": transformer.parameters.id,
11521153
"max_loading": transformer.max_loading.m,
11531154
"sn": transformer.sn.m,
1155+
"tap": transformer.tap,
11541156
"geom": transformer.geometry.__geo_interface__ if transformer.geometry is not None else None,
11551157
}
11561158

0 commit comments

Comments
 (0)