zsylvester / meanderpy Goto Github PK

Is it possible to have Radius of Curvature stats be automatically calculated?

Hello,

I got an error when using meanderpy module. I used python 3.6, meanderpy v0.1.6, and numpy v1.18.4.

It seems to have a bug at line 487 when it is trying to get an integer value but instead got a float (1+s[-1]/deltas). A single slash is deprecated to do floor division under python 3.

I tried to change the code as unew = np.linspace(0,1,1+int(s[-1])//int(deltas)) or unew = np.linspace(0,1,1+int(s[-1]/deltas))and it seems to work. If this is the right way to solve the problem, I can create a pull request for bug fix. Thank you!

When I'm running the channel migration is giving me back this type of error as shown in the attached picture. However, I've tried to fixe my numpy version, but it still crashing.

Running meanderby example notebook fails at the following on python 3:

ValueError                                Traceback (most recent call last)
<ipython-input-17-1cd93eb068e1> in <module>()
----> 1 chb_3d = chb.build_3d_model('submarine',h_mud=1.0,levee_width=3000.0,h=10.0,w=W,bth=10.0,dcr=10.0,dx=20.0,delta_s=deltas,starttime=chb.cl_times[50],endtime=chb.cl_times[55])

/content/meanderpy/meanderpy/_meanderpy.py in build_3d_model(self, model_type, h_mud, levee_width, h, w, bth, dcr, dx, delta_s, starttime, endtime)
    346         for i in range(0,2):
    347             pt = np.asarray(plt.ginput(1))
--> 348             pts[i,:] = pt
    349             plt.scatter(pt[0][0],pt[0][1])
    350         plt.plot([pts[0,0],pts[1,0],pts[1,0],pts[0,0],pts[0,0]],[pts[0,1],pts[0,1],pts[1,1],pts[1,1],pts[0,1]],'r')

ValueError: could not broadcast input array from shape (0) into shape (2)

Hello Zoltán,

I was trying to work with the Google collab instance to see about using it in my Sedimentology course. I ran into an issue.

chb.migrate(nit,saved_ts,deltas,pad,crdist,Cf,kl,kv,dt,dens,t1,t2,t3,aggr_factor) # channel migration
TypeError: migrate() missing 1 required positional argument: 'aggr_factor'

Near as I can tell, it is because there is now a Depths argument, so aggr_factor is not in the right position. I am also unsure of what to enter for a Depths argument either, but haven't looked at further examples of the code. I have minimal python experience.

Thanks,

Ben

Hi Zoltán,

In the notebook, for the fluvial model you set dcr to 10, is this a necessary parameter for fluvial models?

Thanks,
Thomas

Sir, I found that you have multiplied a new coefficient after calculating the mobility R1. This new coefficient seems to be the -2/3 power of the total length of the river divided by the total length of the x dimension. Where does this coefficient appear in the paper?