Skip to content
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

Closed
5 tasks done
brownag opened this issue Jun 2, 2021 · 5 comments
Closed
5 tasks done

fetchOSD upgrades #186

brownag opened this issue Jun 2, 2021 · 5 comments
Labels
SoilWeb Related to the SoilWeb / UCDavis LAWR servers, Henry Mount DB, backend functionality and public API

Comments

@brownag
Copy link
Member

brownag commented Jun 2, 2021

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}

    • should work with official URL, custom URL/mirror, and local files; useful for checking text-based products off cycle from SKB refresh
  • Re-factor downloadParseSave() code to use OSD JSON files produced by {SoilKnowledgeBase}

    • New {SoilKnowledgeBase} OSD products to be created by SKB create_OSD.R in OSD folder:
      • parsed site and horizon data tables are included in main JSON result
  • Set up get_OSD() as interface for JSON, HTML or TXT sources of OSD data

  • Incorporate 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#25

@dylanbeaudette
Copy link
Member

See preliminary re-factored code in R/OSD-file-access.R. Code doesn't have to stay here, just a convenient parking place.

@brownag
Copy link
Member Author

brownag commented Jul 12, 2021

Consolidated the OSD file access into a new method get_OSD() which works on local or remote JSON, HTML and TXT sources of OSDs.

The following items are outside the scope of soilDB-specific updates so they have been removed from the task list for this issue.

  • table/identifiers of missing colors predicted via OLS(value, chroma)
  • "complete" full-text .sql file for full-text search
  • "sectionized" full-text .sql file for full-text search

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 OSD_query() so the methods used to generate those files do not need to be addressed in this issue.

@brownag brownag added the SoilWeb Related to the SoilWeb / UCDavis LAWR servers, Henry Mount DB, backend functionality and public API label Jul 23, 2021
@dylanbeaudette
Copy link
Member

  • need to be able to splice in "inferred" data (such as colors predicted by OLS) for parity with old result

This was added to the pre-processing code, SoilWeb API, and fetchOSD() code some time back.

@dylanbeaudette
Copy link
Member

  • Incorporate JSON data sources into fetchOSD()

Now that we have get_OSD, I think that we can safely make this item as "done", and close the issue--yeah?

@brownag
Copy link
Member Author

brownag commented Oct 28, 2021

  • Incorporate JSON data sources into fetchOSD()

Now that we have get_OSD, I think that we can safely make this item as "done", and close the issue--yeah?

Yes agreed. It is as integrated as it is going to be from perspective of soilDB.

@brownag brownag closed this as completed Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SoilWeb Related to the SoilWeb / UCDavis LAWR servers, Henry Mount DB, backend functionality and public API
Projects
None yet
Development

No branches or pull requests

2 participants