lter / lterwg-som-shiny

Shiny app of the LTER SOM working group
2 stars 1 forks source link

improve depth filter handling of O-horizon #30

Closed piersond closed 4 years ago

piersond commented 4 years ago

Do we currently have negative values for O-horizons? Or are they denoted with top and bottom depth of zero? How can we modify the sliders to improve handling of these layers?

wwieder commented 4 years ago

This is not done consistently. Some times there's a horizon character string that help (when depth_top = 0). I've also put a threshold of lyr_soc < 20%C or bd_samp > 0.6 to screen out what are likely organic horizons. Once again, no perfect solutions...

On Mon, Oct 7, 2019 at 2:24 PM Derek Pierson notifications@github.com wrote:

Do we currently have negative values for O-horizons? Or are they denoted with top and bottom depth of zero? How can we modify the sliders to improve handling of these layers?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/lter/lterwg-som-shiny/issues/30?email_source=notifications&email_token=AB5IWJAFVSCIH3FSXNDVTG3QNOLGVA5CNFSM4I6JLHJ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HQFNGBA, or mute the thread https://github.com/notifications/unsubscribe-auth/AB5IWJAVQ3ZY2PUOFL4MLCDQNOLGVANCNFSM4I6JLHJQ .

-- Will Wieder Project Scientist CGD, NCAR 303-497-1352

piersond commented 4 years ago

Changed depth slider min and max values to -3 to 300. Also changed sliders to take a value range. We'll have to see how we like using the ranges. It's slightly more complicated, but allows for more exact selection of soil layers.