Skip to content

Commit

Permalink
Update river_flow_dynamics.py
Browse files Browse the repository at this point in the history
scipy problems with sp.sparse.linalg.cg
nox test require tol as argument to pass the test.

pytest uses rtol.

We removed that argument and left it with the default value of 1e-5.
  • Loading branch information
angelmons committed Sep 4, 2024
1 parent de899ae commit 63effb6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions landlab/components/river_flow_dynamics/river_flow_dynamics.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@
>>> flow_depth = np.reshape(grid['node']["surface_water__depth"],(nRows,nCols))[10,:]
>>> np.round(flow_depth,2)
>>> np.round(flow_depth,3)
array([0.5 , 0.491, 0.48 , 0.473, 0.467, 0.464, 0.46 , 0.458, 0.455,
0.454, 0.452, 0.45 , 0.449, 0.448, 0.446, 0.445, 0.443, 0.442,
0.441, 0.439, 0.438, 0.437, 0.435, 0.434, 0.433, 0.431, 0.43 ,
Expand Down Expand Up @@ -1617,7 +1617,7 @@ def run_one_step(self):
left_hand_side,
right_hand_side,
M=Mi,
tol=self._pcg_tolerance,
#rtol=self._pcg_tolerance,
maxiter=self._pcg_max_iterations,
atol=0,
)
Expand Down

0 comments on commit 63effb6

Please sign in to comment.