agilescientific / modelr-app

Modelr front end
Apache License 2.0
0 stars 1 forks source link

Fix horizontal and vertical log plot scaling and limits #156

Closed EvanBianco closed 9 years ago

EvanBianco commented 9 years ago

Both the log curves and seismic trace wiggles need to have the same vertical and horizontal scales for the subbed and unsubbed fluid cases.

To illustrate the issue, consider the density log in the example shown here.

image

The fluid substitution density log (dashed blue), should completely line up with the unsubbed plot, but it fails for two reasons: The time-scale range is longer, which compresses a longer curve to fit in the plot window, and the x-range of the plot is normalized to a different extreme value.

I would recommend setting the plot extents based on the unsubstituted log, which might mean chopping off bottom of the substituted log, because it's length will be longer.

Same thing needs to happen for logs and the wiggle traces; x's and y's.

ben-bougher commented 9 years ago

Good test case. It looks like there is a bug with the first layer not having fluids defined. It isn't a plotting bug, we are actually calculating different values in the first layer, which is why they don't line up.

The y-scales shouldn't line up, as we always show 1000 m of data. Since the velocities are different we get different TWT extents. I can pad out the shorter one with zeros or a constant value to be the same length to to make things look symmetric.

ben-bougher commented 9 years ago

Everything should be the same length and correct amplitude.