tcoulthard / caesar-lisflood

model
2 stars 2 forks source link

Wilcock and Crowe sediment transport calculation wrong #10

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
There is an error in how CL calculates D50 - which is a component of the 
wilcock and crowe sediment tranport formulae. This will lead to slightly 
different totals in sediment exported and some topographic differences in where 
erosion and deposition is occurring. 

This only effects you if you are using W&C for sediment transport.

The problem can be rectified by changing line 6068 from:

Dfifty = Math.Exp(min + ((max - min) * ((cum_tot[i] - (active_thickness * 0.5)) 
/ (cum_tot[i] - cum_tot[i - 1]))));

to:

Dfifty = Math.Exp(max - ((max - min) * ((cum_tot[i] - (active_thickness * 0.5)) 
/ (cum_tot[i] - cum_tot[i - 1]))));

I'll post up an updated version of the .exe and source code.

Original issue reported on code.google.com by Tom.Coul...@gmail.com on 10 Oct 2012 at 4:49

GoogleCodeExporter commented 9 years ago
This has now been fixed, and updated code placed on the downloads tab. Older 
versions have been deprecated. 

Original comment by Tom.Coul...@gmail.com on 10 Oct 2012 at 5:00

GoogleCodeExporter commented 9 years ago

Original comment by Tom.Coul...@gmail.com on 6 Nov 2013 at 3:39