Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updates to make Chapter 15 in Biodigs book work oob #1

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

almahmoud
Copy link

@almahmoud almahmoud commented Jul 10, 2024

Planning on using the BioDigs Chapter 15 (https://hutchdatascience.org/GDSCN_BioDIGS_Book/exploring-soil-testing-data-with-r.html) as a quick introduction to R for students in the YES4CURE program as discussed with Natalie and Ava.

  1. When running through it, there was a minor bug in getting the data, due to the fact that gsheet2tbl seems to replace spaces and punctuation with ., so I updated the code to expect that sometimes it's replacing with dots, sometimes it's keeping spaces, not entirely sure why. Turning PR into draft since this isn't consistent like I originally assumed.
  2. The material uses tidyverse in Part 2 but didn't have it as an explicit dependency, so I added that to the DESCRIPTION file.

We can run from the fork for the students, so no urgency in looking at this, and no need to merge if seems inappropriate, but I thought it would be beneficial to PR upstream in case helpful!

I didn't make any version bump as I wasn't sure about your preferred versioning scheme.

These were tested with R 4.4.1, sessionInfo below for list of package versions

other attached packages:
 [1] lubridate_1.9.3        forcats_1.0.0          stringr_1.5.1          purrr_1.0.2           
 [5] readr_2.1.5            tibble_3.2.1           ggplot2_3.5.1          tidyverse_2.0.0       
 [9] dplyr_1.1.4            tidyr_1.3.1            gsheet_0.4.5           BioDIGSData_0.0.0.9000

loaded via a namespace (and not attached):
 [1] utf8_1.2.4        generics_0.1.3    stringi_1.8.4     hms_1.1.3         digest_0.6.36    
 [6] magrittr_2.0.3    grid_4.4.1        timechange_0.3.0  pkgload_1.3.4     fastmap_1.2.0    
[11] processx_3.8.4    pkgbuild_1.4.4    sessioninfo_1.2.2 urlchecker_1.0.1  ps_1.7.7         
[16] promises_1.3.0    httr_1.4.7        fansi_1.0.6       scales_1.3.0      cli_3.6.3        
[21] shiny_1.8.1.1     rlang_1.1.4       munsell_0.5.1     ellipsis_0.3.2    withr_3.0.0      
[26] remotes_2.5.0     cachem_1.1.0      devtools_2.4.5    tools_4.4.1       tzdb_0.4.0       
[31] memoise_2.0.1     colorspace_2.1-0  httpuv_1.6.15     curl_5.2.1        vctrs_0.6.5      
[36] R6_2.5.1          mime_0.12         lifecycle_1.0.4   fs_1.6.4          htmlwidgets_1.6.4
[41] usethis_2.2.3     miniUI_0.1.1.1    pkgconfig_2.0.3   desc_1.4.3        callr_3.7.6      
[46] gtable_0.3.5      pillar_1.9.0      later_1.3.2       glue_1.7.0        profvis_0.3.8    
[51] Rcpp_1.0.12       tidyselect_1.2.1  rstudioapi_0.16.0 xtable_1.8-4      htmltools_0.5.8.1
[56] compiler_4.4.1 

@almahmoud almahmoud marked this pull request as draft July 11, 2024 02:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant