ncss-tech / sharpshootR

Miscellaneous soil data management, summary, visualization, and conversion utilities to support soil survey.
http://ncss-tech.github.io/sharpshootR/
17 stars 3 forks source link

re-think hillslopeProbability, surfaceShapeProbability, geomPosHillProbability, geomPosMountainProbability #12

Closed dylanbeaudette closed 2 years ago

dylanbeaudette commented 6 years ago

These functions are currently broken, due to a minor SQL syntax error.

Error in SDA_query(q) : 
Invalid query: The multi-part identifier "component.cokey" could not be bound.
The multi-part identifier "component.cokey" could not be bound.

SoilWeb has pre-cached versions of these data available for hillslope, geomorphic component, and mountain slope. A couple of options:

Major differences in implementation between SoilWeb cached proportions and SDA/R on-the-fly:

More importantly: should proportions be based on acres-mapped or number of occurrences? Incorporating acres-mapped is much more efficiently done in a big batch (e.g. SoilWeb version), but may be possible with SDA if done one series at a time.

Once this is sorted out, we can start making figures like this:

image

dylanbeaudette commented 6 years ago

SDA/SQL syntax error corrected in 885e14ac10fc1838b09d43a5bec49620b4b80900. Still need to figure out long-term interface to these data.

dylanbeaudette commented 6 years ago

Related issue in soilDB: https://github.com/ncss-tech/soilDB/issues/60

dylanbeaudette commented 5 years ago

Note that soilDB::fetchOSD() returns two of the four summaries.

dylanbeaudette commented 2 years ago

Deprecated and slated for removal in the next major release. Need to add the surface shape morph to fetchOSD.