search

lasiadhi / USACE-Bathymetry

0 stars 2 forks source link

wave height plot(H&index) in forward file #126

Closed montymudra closed 8 years ago

montymudra commented 8 years ago

The modified forward model seems not giving the correct plot for wave height.

montymudra commented 8 years ago

In code->forward-> realdata_H_modified.m waveheight_H_modified.m plot is in png file as H&index

thesser1 commented 8 years ago

Why is there a rho_g/8 in the equation on line 64? It makes you divide the dissipation value by an extra 1/8_rho*g in the solution for H.

On Mon, Jul 25, 2016 at 9:05 PM, Monty notifications@github.com wrote:

Assigned #126 https://github.com/lasiadhi/USACE-Bathymetry/issues/126 to @thesser1 https://github.com/thesser1.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lasiadhi/USACE-Bathymetry/issues/126#event-734344497, or mute the thread https://github.com/notifications/unsubscribe-auth/ACmnbLQiffEUlX1KSGHPBZA_Swm48KTjks5qZV1sgaJpZM4JUsXC .

thesser1 commented 8 years ago

Sorry, I don't have access to fsolve in my matlab so I cannot test this on my computer. Let me know if it worked.

montymudra commented 8 years ago

that c is just the constant for finite difference steps. @dwb1988 will update the file writing the coefficient. cc is celerity in that code.

montymudra commented 8 years ago

file is pushed.

dwb1988 commented 8 years ago

changed 'c' to 'coe' tried two ways to get celerity as 'cc' (2_pi/(k(i)_Tb))& 'cc1' (sqrt(g*h(i))), slightly difference showed not sure how to deal with the part where depth down to zero (just forced 'H' to zero) the eqn gives NaN for H around i=106 but h is not that small (0.0025).

thesser1 commented 8 years ago

Just made a push, please try again. I think the problem was on my teaching so I apologize with regards to the Hrms but also make sure you are not dividing by a constant that does not match the equation.

dwb1988 commented 8 years ago

It works and the trend changes a little (still not look like in a good shape)

plot in 'forward' file as those 'png's

thesser1 commented 8 years ago

I think the images look like solutions for the bathymetry and the wave characteristics you are running. Go with it.

On Mon, Jul 25, 2016 at 11:33 PM, dwb1988 notifications@github.com wrote:

It works and the trend changes a little (still not look like in a good shape)

plot in 'forward' file as those 'png's

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lasiadhi/USACE-Bathymetry/issues/126#issuecomment-235153564, or mute the thread https://github.com/notifications/unsubscribe-auth/ACmnbJiJ01H58C99N0RTygJouwyT9Ecnks5qZX_2gaJpZM4JUsXC .