-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fetchOSD
upgrades
#186
Comments
See preliminary re-factored code in |
Consolidated the OSD file access into a new method The following items are outside the scope of soilDB-specific updates so they have been removed from the task list for this issue.
The OSD JSON parsing method and new structure is robust enough to add new data at site or horizon level and associated identifiers as needed. The .sql files for full-text search are used only for SoilWeb backend of |
This was added to the pre-processing code, SoilWeb API, and |
Now that we have |
Yes agreed. It is as integrated as it is going to be from perspective of soilDB. |
Merging functionality w/ @dylanbeaudette (https://github.com/ncss-tech/parse-osd/) for weekly rebuilds of OSD-related data snapshots via https://github.com/ncss-tech/OSDRegistry and https://github.com/ncss-tech/SoilKnowledgeBase
Move
get_OSD(url, what = c("txt","html"))
routines from parse-osd repo to {soilDB}Re-factor
downloadParseSave()
code to use OSD JSON files produced by {SoilKnowledgeBase}Set up
get_OSD()
as interface for JSON, HTML or TXT sources of OSD dataIncorporate JSON data sources into
fetchOSD()
need to be able to splice in "inferred" data (such as colors predicted by OLS) for parity with old result
For further discussion on source data used by
OSD_query()
and other OSD-derived products see ncss-tech/SoilKnowledgeBase#25The text was updated successfully, but these errors were encountered: