Conversation
chore: bump R package version pkgver
BREAKING CHANGE wip: more stringsAsFactors fixes fix(SSURGO_spatial_query): eliminate side effects on user options fix: uncode
- BREAKING CHANGE
- BREAKING CHANGE
- BREAKING CHANGE
…, add TODO for "classifier" conflict
- BREAKING CHANGE
- BREAKING CHANGE
… "get_veg_data_from_NASIS_db" superseded by `fetchVegdata()`
This reverts commit 4e2bb50.
|
The new SDA vignette is excellent, thanks for taking the time to write this. I'm glad that you included a section on chunk-wise processing. Consider using the following instead of a for loop: chunks <- makeChunks(mukey)
res <- lapply(split(mukey, chunks), ... )For most use cases it doesn't really matter, but in my experience, the above approach is readily converted to parallel processing or related functions like Also, it might be useful to demonstrate iteration over soil survey area symbols. I'd be happy to add that later on as it is not a high priority--just something that I find myself doing all the time. |
|
I was thinking that a for loop was easier to understand, but I personally prefer |
…="MIN" or "MAX" - this has always worked historically, but the recent attempt to provide more informative error message omitted these
|
Winbuilder is taking forever today/yesterday, I got one through last night but it took several hours, it was just one NOTE about URL issues fixed above. I am going to go ahead and merge this shortly. I am not in a huge rush to get it on CRAN, but want it available for the stats class next week as removing all the deprecation flags will make it easier for folks to see other output from e.g. |
This will be a significant minor release of soilDB that removes several deprecated aspects of result structure, function arguments, etc.
stringsAsFactorsusagesReturn original NASIS record IDs from SDM in relevant SDA queries Return original NASIS record IDs from SDM in relevant SDA queries #397Full resolution of issue 397 has been deferred pending SSURGO schema updates or new routines for amending schemaWill close #242, #375, and #424