JustinGOSSES / wellioviz

d3.js v5 visualization of well logs
https://justingosses.github.io/wellioviz/#introduction
Apache License 2.0
49 stars 12 forks source link

Fix explicit depths (max and min) so it works on multiple levels #50

Open JustinGOSSES opened 4 years ago

JustinGOSSES commented 4 years ago

Describe the bug

max_depth and min_depth doesn't quite work as it should

Expected behavior A clear and concise description of what you expected to happen.

_A max_depth key:value refers to the depth that is used as the plotting max depth no matter the actual deepest depth found in a curve/well._

For example, if there was a well where the curves all went to 200 but the max depth was set to be 180, the curves would only be plotted to 180.

Another example, if there was a well where all the curves had a deepest point of 120 but the max depth was 180, then the plot would show empty space below where the curve lines were all the way down to 200.

This is not the same as "top_depth" or "bottom_depth" used to describe how to create a depth curve along with the interval. For example, top = 100, bottom = 200, and interval = 5.

There should be a way to set max_depth and min_depth at different levels of organization that then flow to smaller levels.

For example, you should be able to set max_depth and min_depth at the curve, curve_box, and cross_section level. Also, any max_depth set at cross_section should be applied to the curve_box and each curve.

This means the code should: