Skip to content

Commit e15ad0c

Browse files
committed
remove TODO from code: either resolved or for future update
1 parent c6864e7 commit e15ad0c

File tree

5 files changed

+0
-10
lines changed

5 files changed

+0
-10
lines changed

pygridsim/core.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,6 @@ def view_source_node(self):
134134
Args:
135135
indices (optional): Which indices to view the nodes at.
136136
If none given, display all
137-
138-
TODO once capability for more source nodes is initialized
139137
"""
140138
source_obj = altdss.Vsource["source"]
141139
source_info = {}
@@ -146,8 +144,6 @@ def view_source_node(self):
146144
def solve(self):
147145
"""
148146
Initialize "solve" mode in AltDSS, then allowing the user to query various results on the circuit
149-
150-
TODO: error handling here
151147
"""
152148
altdss.Solution.Solve()
153149

@@ -170,7 +166,6 @@ def clear(self):
170166
Must call after we are done using the circuit, or will cause re-creation errors.
171167
172168
We only work with one circuit at a time, can only have one PyGridSim object at a time.
173-
TODO: maybe this isn't necessary because it's done in the beginning
174169
"""
175170
altdss.ClearAll()
176171
self.num_loads = 0

pygridsim/enums.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
from enum import Enum
22
import pygridsim.defaults as defaults
33

4-
# todo: update to have a name so that users can query from name
54
class SourceType(Enum):
65
TURBINE = "turbine"
76
POWER_PLANT = "powerplant"

pygridsim/parameters.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ def random_param(range):
2727
[lower_bound, upper_bound]; range of typical value
2828
Return:
2929
Randomly selected value in range
30-
TODO: allow for non-uniform distributions
3130
"""
3231
if type(range) is not list:
3332
return range
@@ -120,7 +119,6 @@ def make_pv(load_node, params, num_panels, count):
120119
pv.Bus1 = load_node
121120
pv.Phases = get_param(params, "phases", defaults.PHASES)
122121
pv.kV = get_param(params, "kV", random_param(defaults.SOLAR_PANEL_BASE_KV) * num_panels)
123-
# todo: inverter capacity?
124122

125123
def make_generator(params, gen_type, count):
126124
"""

pygridsim/results.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ def query_solution(query):
1111
1212
Args:
1313
query: a query for the solve function
14-
TODO: only BusVMag, Losses, TotalPower is supported, need to make accessible which queries are supported
1514
Return:
1615
Query result or the string "Invalid" if the query is not supported
1716
"""

tests/test_circuit.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,6 @@ def test_006_update_multiple_source(self):
9898
circuit.add_lines([("source", "load0")], "HV")
9999
circuit.solve()
100100
print(circuit.results(["Voltages"]))
101-
# TODO: can add assert to make sure it's in reasonable range?
102101

103102
def test_007_export(self):
104103
circuit = PyGridSim()

0 commit comments

Comments
 (0)