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

Refactor SoilGrids functions #3416

Open
dlebauer opened this issue Jan 17, 2025 · 1 comment · May be fixed by #3455
Open

Refactor SoilGrids functions #3416

dlebauer opened this issue Jan 17, 2025 · 1 comment · May be fixed by #3455
Labels
ccmmf issues and pre related to the ccmmf project

Comments

@dlebauer
Copy link
Member

dlebauer commented Jan 17, 2025

Description

This is motivated by

Changes will:

  • Improve the efficiency and scalability of SoilGrids queries, especially for large datasets.
  • Simplify and generalize the workflow to handle multiple SoilGrids variables.

Proposed Solution

Refactor soilgirds_soilC_extract and soilgrids_texture_extraction into three modular functions to increase functionality and flexibility, generalized to work with any variable and depth layer.

  • download_soilgrids_raster a function to download tiff files
  • query_soilgrids a function to query data from points or polygons from remote VRTs or from local rasters
  • soilgrids_uncertainty to estimate uncertainties
  • soilgrids_carbonstocks to calculate OCS to any depth from SOC (to generalize soilgrids 0-30 OCS data product to multiple depths)

Alternatives Considered

Status Quo

FYI @mdietze @DongchenZ @Qianyuxuan @infotroph

@dlebauer dlebauer added the ccmmf issues and pre related to the ccmmf project label Feb 12, 2025
@AritraDey-Dev
Copy link
Contributor

@dlebauer would it help #3455 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ccmmf issues and pre related to the ccmmf project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants