Closed msleckman closed 2 years ago
@lekoenig Changes incorporated and ready for merge.
merge conflict resolved. After incorporating the latest changes from USGS-R/main
there's an issue with the manual download of study_monitoring_sites.zip
(1_fetch.R:L46). OK to merge, trouble shooting will take place in #10.
Pipeline runs successfully for me with latest merge from main.
This PR is ready for review.
Added 3 targets:
p1_selected_statsgo_sbid_children
: Pulls the specific child items from the Statsgo soil characteristics data release from Science base. Only interested in Texture attributes and Soil Layer attributes. (intermediate target)p1_download_statsgo_text_layer_attr
: file target downloading the statsgo zip files (intermediate target)p1_statsgo_soil_df
: reads in and scopes statsgo dataset + cbinds so that we have 1 target df object with total upstream (TOT), accumulated (ACC), and catchment (CAT)-level values. (final fetch target)Added 1 function:
sb_read_filter_by_comids()
outputsp1_statsgo_soil_df
. Built for statsgo but generalizable to any comid-level sb dataset that requires fetching and initial scoping (col selection and aoi row filtering).p1_statsgo_soil_df
output:Notes of
p1_statsgo_soil_df
target:c("KFACT","KFACT_UP","NO10AVE", "NO4AVE","SILTAVE","CLAYAVE","SANDAVE","WTDEP")
(see attr description in abstract and metadata for layer attr and texture attr). Please let me know if there are any other variables we want