Skip to content

Commit 629cecf

Browse files
mjrenomjreno
authored andcommitted
add todo comments
1 parent fb46588 commit 629cecf

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

docs/examples/twri.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -243,19 +243,17 @@ def plot_head(head, workspace):
243243
# recharge
244244
recharge = np.repeat(np.expand_dims(LAYER_NODATA, axis=0), repeats=nper, axis=0)
245245
recharge[0, ...] = 3.0e-8
246-
# recharge[0, 0, 0] = 3.0e-7
247-
# rch = flopy4.mf6.gwf.Rcha(irch=1, recharge=recharge.reshape(nper, -1), dims=dims)
248246
rcha = flopy4.mf6.gwf.Rcha(recharge=recharge.reshape(nper, -1), dims=dims)
249247

250248
# remove list based inputs
249+
# TODO: show variations on removing packages
251250
gwf.chd.remove(chd)
252-
# del gwf.chd[0]
253251
del gwf.drn[0]
254252
del gwf.wel[0]
255253
del gwf.rch[0]
256254

257255
# add array based inputs
258-
# gwf.chdg = [chdg]
256+
# TODO: needs type checking and list consolidation support (see comments in gwf init)
259257
gwf.chd = [chdg]
260258
gwf.drng = [drng]
261259
gwf.welg = [welg]

flopy4/mf6/gwf/__init__.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,11 @@ def budget(self):
9797
oc: Oc | None = field(block="packages", default=None)
9898
npf: Npf | None = field(block="packages", default=None)
9999
sto: Sto | None = field(block="packages", default=None)
100+
# TODO: implement type check for all lists (and singletons?)
100101
chd: list[Chd] = field(block="packages")
101102
chdg: list[Chdg] = field(block="packages")
102-
# chd: List[Union[Chd, Chdg]] = field(block="packages", factory=List)
103+
# TODO: consolidate all package flavors to single list
104+
# based on hydrologic feature
103105
# chd: List[Union[Chd, Chdg]] = field(block="packages")
104106
drn: list[Drn] = field(block="packages")
105107
drng: list[Drng] = field(block="packages")

0 commit comments

Comments
 (0)