Skip to content

Commit

Permalink
fix: bug where topo diffusion happened after mud
Browse files Browse the repository at this point in the history
documentation says that topo diffusion should happen after sand 
deposition, but before mud deposition.
  • Loading branch information
ericbarefoot committed May 13, 2020
1 parent 2c5fed3 commit cba1866
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions pyDeltaRCM/sed_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,11 @@ def sed_route(self):
self.Vp_dep_mud[:] = 0

self.sand_route()
self.mud_route()

self.topo_diffusion()

self.mud_route()

def deposit(self, Vp_dep, px, py):
"""deposit sand or mud"""

Expand Down Expand Up @@ -200,8 +201,6 @@ def sed_parcel(self, theta_sed, sed, px, py):
if ind[0] == 0:
weight[0, :] = 0

# return np.cumsum(weight.flatten())

new_cell = self.random_pick(np.cumsum(weight.flatten()))

jstep = self.iwalk.flat[new_cell]
Expand Down Expand Up @@ -256,7 +255,6 @@ def sand_route(self):

self.sed_parcel(theta_sed, 'sand', px, py)

# self.topo_diffusion()
def topo_diffusion(self):
"""
Diffuse topography after routing all coarse sediment parcels
Expand Down

0 comments on commit cba1866

Please sign in to comment.