Skip to content

soilDB 2.9.0#426

Merged
brownag merged 53 commits intomasterfrom
rdep41
Jan 23, 2026
Merged

soilDB 2.9.0#426
brownag merged 53 commits intomasterfrom
rdep41

Conversation

@brownag
Copy link
Member

@brownag brownag commented Dec 5, 2025

This will be a significant minor release of soilDB that removes several deprecated aspects of result structure, function arguments, etc.

Will close #242, #375, and #424

@brownag brownag marked this pull request as ready for review January 21, 2026 21:57
@dylanbeaudette
Copy link
Member

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 purrr::map() that have some convenient features (progress bars and such).

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.

@brownag
Copy link
Member Author

brownag commented Jan 22, 2026

I was thinking that a for loop was easier to understand, but I personally prefer lapply() and related for most things. I've made the suggested changes, and changed the example that only had 10 mukeys to a soil survey area example. Also, added a new section on error handling and retry logic.

@brownag
Copy link
Member Author

brownag commented Jan 23, 2026

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. fetchNASIS(). Will tell folks to update their soilDB as part of the precourse tomorrow and initial materials on Monday. I have rebuilt the book with 2.9.0 so I think we are safe to go ahead with that, but if we aren't we'll find out soon and deal with it then.

@brownag brownag merged commit 8287ef7 into master Jan 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

soilDB minimum R version terra usage is not robust to missing/invalid GDAL/PROJ installation ensure soilDB uses NASIS/SDA column names at all times

2 participants