wellioviz icon indicating copy to clipboard operation
wellioviz copied to clipboard

Put in architecture for having multple depths but not finalized so it always looks nice... needs work

Open JustinGOSSES opened this issue 5 years ago • 4 comments

Change how depths are handled to ensure you can have multiple depth curves per well.

JustinGOSSES avatar Dec 13 '19 23:12 JustinGOSSES

  • Change DepthCurve name to be associated with another well log curve
  • Have it be an array of depth curve names
  • Have it be okay to just be one depth curve name, even if there are multiple curve names
  • Have the curvebox2 code be able to find the max and min across different well log curves

Mostly need to change :

  • TEMPLATE
  • CURVEBOX2 function

JustinGOSSES avatar Dec 14 '19 00:12 JustinGOSSES

NOTES: Various sizes & positions impact getting y axis to look nice with both single and double depth margin.left ~ has been 60 pixel value yaxis translate value ~ translate(${margin.left*.333},0)) yaxis text appended translate value ~ .attr("y", 0 - (margin.left*0.6)) yaxis text size & number size ~ ~ .attr("dy", ".75em")

  • Would be nice in some ways if margins on left side where y axes are just changed when you selected to show two depths, but that would probably end up being a huge pain for someone who didn't work that exact sizing behavior, so not the greater way. Better to require developers to be explicit with everything.

JustinGOSSES avatar Feb 25 '20 14:02 JustinGOSSES

add new data type "reference_track" and handle in there Make visual distinction of when "real" depth track ends (exaggerated example below)

image

kernant avatar Feb 26 '20 14:02 kernant

Have this working so I know the architecture fine but not exposed to users yet as I need to add some additional options for CSS sizing/positioning. Putting slightly lower in priority as I know architecture is fine now.

JustinGOSSES avatar Mar 09 '20 03:03 JustinGOSSES