trashbirdecology / bbsassistant Goto Github PK
View Code? Open in Web Editor NEWAn R package for downloading and handling data and information from the North American Breeding Bird Survey.
License: Creative Commons Zero v1.0 Universal
An R package for downloading and handling data and information from the North American Breeding Bird Survey.
License: Creative Commons Zero v1.0 Universal
Writeup for JOSS submission
Saving example data using usethis::use_data()
will allow us to avoid users having to download files from the internet AND avoid directory creation. This also allows us to run tests in CI.
Ex:
x<-1
y<-2
usethis::use_data(x, y)
E.g., import GIS datafiles +meta from BBS
Describe the bug
The example in the help file for get_bbsData
uses an out of date function name (getDataBBS
): https://github.com/TrashBirdEcology/bbsAssistant/blob/master/R/get_bbsData.R#L25
To Reproduce
See ?bbsAssistant::get_bbsData
Expected behavior
The function name in the example is getDataBBS
, but I expected it to be get_bbsData
.
Session info
─ Session info ──────────────────────────────────────
setting value
version R version 3.6.1 (2019-07-05)
os Ubuntu 18.04.3 LTS
system x86_64, linux-gnu
ui RStudio
language en_US
collate en_US.UTF-8
ctype en_US.UTF-8
tz America/Denver
date 2019-08-27
─ Packages ──────────────────────────────────────────
package * version date lib source
assertthat 0.2.1 2019-03-21 [1] CRAN (R 3.6.0)
bbsAssistant * 1.0.0 2019-08-27 [1] local
cli 1.1.0 2019-03-19 [1] CRAN (R 3.6.0)
crayon 1.3.4 2017-09-16 [1] CRAN (R 3.6.0)
magrittr 1.5 2014-11-22 [1] CRAN (R 3.6.0)
packrat 0.5.0 2018-11-14 [1] CRAN (R 3.6.0)
rstudioapi 0.10 2019-03-19 [1] CRAN (R 3.6.0)
sessioninfo 1.1.1 2018-11-05 [1] CRAN (R 3.6.0)
withr 2.1.2 2018-03-15 [1] CRAN (R 3.6.0)
[1] /home/max/R/x86_64-pc-linux-gnu-library/3.6
[2] /usr/local/lib/R/site-library
[3] /usr/lib/R/site-library
[4] /usr/lib/R/library
Additional context
PR on the way!
I figured I would start an issue thread here for the JOSS Review openjournals/joss-reviews#1768
My first set of comments (1) are more or less "must-haves" to meet the JOSS requirements while the second set (2) are "nice-to-have" suggestions.
paper.md
should probably have an explicit citation for the rBBS
package.
The testthat
tests in test-get_regions.R
fails for me with error:
Error in get_regions() : could not find function "get_regions"
The stated goal of the package is to make working with bbs data more user friendly by consolidating "the efforts of the data user by automating downloading and decompression of .zip data files, downloading route-level information, and saving them as .feather files for speedy import from disk."
As a new user, there were some areas that I thought could be even more user friendly:
The biggest was the manual download > export > import process. Current operation is download to temporary file then manually export to "permanent" disk then later import this data. What if this was revised so that a user could optionally pass a permenent location to get_bbsData
, this location would checked for an existing file (to avoid re-downloading existing files) and automatically exported?
I wish the docs included some maps. For instance, as a new user I didn't know the difference between Region=="S05"
and Region.Code=="FLA"
I felt like some of the calls to the stringr
package in the vignette were not needed. Maybe people who don't know stringr
would have an easier time if these were replaced with base R code? Maybe the region_code
object could be cleaned up so that someone could run region_codes[region_codes$State == "Florida", "zip_states"]
?
The vignette/package uses the feather package AND variables called feather
. This could be confusing.
Maybe the vignette could use a more descriptive title rather than "vignettes"?
I had trouble understanding the subset_speciesList
function. Why does it remove the specified family rather than include it like dplyr::filter
? What does this function offer
beyond dplyr::filter
?
Why are the first and second figures in the vignette different? Seems like they are both plotting AOU.Number=='s06882'
and Region.Code=="S05"
by year.
Seems like the pkgdown
site is out of date. The current landing page doesn't match the README.
Vignette images on the pkgdown site are broken for me...
The vignette chunk Subsetting the BBS count data by AOU number
prints A LOT of output. Maybe it would be good to pass the output through glimpse
like the other chunk outputs?
Describe the bug
The name of the zip file for Wisconsin's data appears to differ between the FTP server (Wiscons.zip
at ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/States), and the data/region_codes.rds
file (Wisons.zip
), causing the get_bbsData()
function to fail when trying to access data for Wisconsin.
To Reproduce
library(bbsAssistant)
data("region_codes")
region_filename <- region_codes$zip_states[region_codes$State == "WISCONSIN"]
print(region_filename)
#> [1] "Wisons.zip"
bbs_data <- get_bbsData(file = region_filename)
#> Warning in download.file(ZipName, temp, quiet = FALSE): cannot open URL
#> 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/States/Wisons.zip':
#> FTP status was '550 Requested action not taken; file unavailable'
#> Error in download.file(ZipName, temp, quiet = FALSE): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/States/Wisons.zip'
Created on 2019-08-16 by the reprex package (v0.3.0)
Expected behavior
Expected the BBS data for Wisconsin to be downloaded.
Session info
─ Session info ─────────────────────────────────────────────────────────────────────────────────────────
setting value
version R version 3.6.1 (2019-07-05)
os Ubuntu 18.04.3 LTS
system x86_64, linux-gnu
ui RStudio
language en_US
collate en_US.UTF-8
ctype en_US.UTF-8
tz America/Denver
date 2019-08-16
─ Packages ─────────────────────────────────────────────────────────────────────────────────────────────
package * version date lib source
assertthat 0.2.1 2019-03-21 [1] CRAN (R 3.6.0)
callr 3.3.1 2019-07-18 [1] CRAN (R 3.6.1)
cli 1.1.0 2019-03-19 [1] CRAN (R 3.6.0)
clipr 0.7.0 2019-07-23 [1] CRAN (R 3.6.1)
crayon 1.3.4 2017-09-16 [1] CRAN (R 3.6.0)
digest 0.6.20 2019-07-04 [1] CRAN (R 3.6.1)
evaluate 0.14 2019-05-28 [1] CRAN (R 3.6.0)
fs 1.3.1 2019-05-06 [1] CRAN (R 3.6.0)
htmltools 0.3.6 2017-04-28 [1] CRAN (R 3.6.0)
knitr 1.24 2019-08-08 [1] CRAN (R 3.6.1)
packrat 0.5.0 2018-11-14 [1] CRAN (R 3.6.0)
processx 3.4.1 2019-07-18 [1] CRAN (R 3.6.1)
ps 1.3.0 2018-12-21 [1] CRAN (R 3.6.0)
R6 2.4.0 2019-02-14 [1] CRAN (R 3.6.0)
Rcpp 1.0.2 2019-07-25 [1] CRAN (R 3.6.1)
reprex 0.3.0 2019-05-16 [1] CRAN (R 3.6.0)
rlang 0.4.0 2019-06-25 [1] CRAN (R 3.6.1)
rmarkdown 1.14 2019-07-12 [1] CRAN (R 3.6.1)
rstudioapi 0.10 2019-03-19 [1] CRAN (R 3.6.0)
sessioninfo 1.1.1 2018-11-05 [1] CRAN (R 3.6.0)
whisker 0.3-2 2013-04-28 [1] CRAN (R 3.6.0)
withr 2.1.2 2018-03-15 [1] CRAN (R 3.6.0)
xfun 0.8 2019-06-25 [1] CRAN (R 3.6.1)
[1] /home/max/R/x86_64-pc-linux-gnu-library/3.6
[2] /usr/local/lib/R/site-library
[3] /usr/lib/R/site-library
[4] /usr/lib/R/library
Additional context
I checked all other regions that have data and this seems to be the only one where the filenames don't match. Maybe also related to the recent BBS data update?
Hey @trashbirdecology! Awesome package. Maybe this is a bug? 🤷♂️
Describe the bug
The get_regions()
function raises an error that seems to be caused by attempting to read a README.txt
file from the FTP server that no longer exists.
To Reproduce
library(bbsAssistant)
regions <- get_regions()
#> Parsed with column specification:
#> cols(
#> X1 = col_character(),
#> X2 = col_character()
#> )
#> Parsed with column specification:
#> cols(
#> X1 = col_character(),
#> X2 = col_character(),
#> X3 = col_character()
#> )
#> Warning in dir.create(codesDir): '/tmp/RtmpHiEJfU/reprex156458487ca2/
#> bbsData/codes' already exists
#> Warning in download.file(url = File, destfile = paste0(codesDir, "/
#> README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/
#> DataFiles/README.txt': FTP status was '550 Requested action not taken; file
#> unavailable'
#> Error in download.file(url = File, destfile = paste0(codesDir, "/README.txt")): cannot open URL 'ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt'
Created on 2019-08-14 by the reprex package (v0.3.0)
Expected behavior
I was expecting to get the region data, like in the README.
Session info
R version 3.6.1 (2019-07-05)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Pop!_OS 18.04 LTS
Matrix products: default
BLAS: /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.7.1
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.7.1
locale:
[1] LC_CTYPE=en_US.UTF-8 LC_NUMERIC=C LC_TIME=en_US.UTF-8 LC_COLLATE=en_US.UTF-8 LC_MONETARY=en_US.UTF-8
[6] LC_MESSAGES=en_US.UTF-8 LC_PAPER=en_US.UTF-8 LC_NAME=C LC_ADDRESS=C LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats graphics grDevices utils datasets methods base
other attached packages:
[1] bbsAssistant_1.0.0
loaded via a namespace (and not attached):
[1] Rcpp_1.0.2 rstudioapi_0.10 whisker_0.3-2 knitr_1.24 magrittr_1.5 hms_0.5.0 here_0.1 R6_2.4.0 rlang_0.4.0
[10] tools_3.6.1 packrat_0.5.0 xfun_0.8 withr_2.1.2 clipr_0.7.0 htmltools_0.3.6 rprojroot_1.3-2 digest_0.6.20 tibble_2.1.3
[19] crayon_1.3.4 processx_3.4.1 callr_3.3.1 readr_1.3.1 ps_1.3.0 vctrs_0.2.0 fs_1.3.1 curl_4.0 zeallot_0.1.0
[28] evaluate_0.14 rmarkdown_1.14 reprex_0.3.0 compiler_3.6.1 pillar_1.4.2 backports_1.1.4 pkgconfig_2.0.2
Additional context
The error seems to stem from this file not existing: ftp://ftpext.usgs.gov/pub/er/md/laurel/BBS/DataFiles/README.txt
Maybe it was removed in the most recent BBS data update?
Include in the function documentation (and readme doc) the significance of the credibility measures, when and where filtering by credibility may be appropriate, and link to relevant pubs.
We need to make sure the get_bbsAous match the AOUs in the actual bbs data.
Further, I propose we trim the LEADING whitespace from get_bbsAous() returned object, and ensure no leading zeroes on the AOU column from the get_bbsData() objects.
I get this warning when running this code. I understand that the file got saved but I was wondering if we could rewrite the warning to something that won't scare or alarm the users. I post the original warning and my suggestion below. In my experience, R users don't always understand errors so I'm always keen on providing more info on the errors as possible so that people may be less freaked out about them. It may be a silly suggestion though. Just a thought.
Original warning (as it is now)
export_bbsFeathers(dataIn = flBBS, newDir = bbsDir, filename = regionFileName.use)
Warning message:
In dir.create(newDir) :
'G:\bbsAssistant package\BBS_test_GP\bbsData' already exists
My suggested warning
Warning message:
In dir.create(newDir) :
'G:\bbsAssistant package\BBS_test_GP\bbsData' already exists. Make sure the new file was saved in this directory: e.g., "Florida.feather"
Prepare for release:
usethis::use_cran_comments()
devtools::check()
devtools::check_win_devel()
Submit to CRAN:
usethis::use_version('major')
cran-comments.md
devtools::submit_cran()
Wait for CRAN...
usethis::use_github_release()
usethis::use_dev_version()
Import and integrate functional group and body mass data into the core bbs raw data.
(From readme.md)
**Users have requested that basic visualization functions be provided. **
This issue was submitted as a task for the 2019 Google Code In
We seek help for creating basic visualization functions for depicting general summary statistics for individual species onto spatial maps (e.g. heat maps similar to this example). For example, a spatial map comprising absolute occurrences (e.g., species A is present or not present at a location-binary) or relative abundances (e.g., a relative heatmap of Species A 'abundance).
Further, we wish to provide the opportunity for the user map basic statistics at multiple spatial scales. For example, mapping at the U.S. state level, at the U.S. county level, or at regional levels (e.g., Bird Conservation Regions).
We are open to suggestions of which spatial projections to use, and which source maps for U.S. and Canadian state and county-level maps (e.g., using Ggmap
vs. a spatial data package within the bbsAssistant
package).
Consider removing all here dependencies throughout the package. I am not sure how here
acts when user installs package and does not work from an .rproj wtihin the package contents.
I.e. it may be best (and more efficient) to remove all calls to here
and instead write the relative directories.
This is recommendation based on my experience while reviewing this package for JOSS (openjournals/joss-reviews#1768).
Is your feature request related to a problem? Please describe.
As a new user of the software I found the API as introduced in the vignette to be a bit complicated. The order of introduction is currently: 1) Create a data directory for storing data; 2) Download and load data into R (this data doesn't end up in the data directory); 3) Export the data into a local file (in a not widely used format; I love feather
but it won't be familiar to many users); 4) Import the data (which requires doing some filename matching and manipulation). This felt like a lot of stuff if my goal is to quickly and easily work the the BBS data.
Describe the solution you'd like
My core use case for a package like this would be "Get BBS data into R and start working with it. Ideally without repeatedly downloading the data if I already have it". With that use case in mind I would consider making get_bbsData
handle all of this for the user, with some optional arguments to control behavior. So, the default behavior for get_bbsData
would be to:
I'd have an argument to allow setting the data directory (where files are checked for and downloaded to) and set a default for this directory to either the working directory (in a clearly named subdirectory) or a .bbsAssistant
directory in the users home directory.
Describe alternatives you've considered
Given that most of the intended usage appears to focus on single BBS files, which are relatively small, another option would be to de-emphasize the feather
storing and loading functionality in the introductory vignette.
Import metadata from various versions and import in an organized fashion.
Describe the bug
There are a couple of minor typos on the README that will raise errors if users follow the instructions as written.
To Reproduce
The instructions under the "Function Descriptions and Vignettes" will lead to two errors:
devtools::build_manual("bbsAssistant)
will raise an error, because there is no closing quote before the last parenthesis. This can be simplified to devtools::build_manual()
, which by default will use the current package directory.
usethis::build_vignettes()
will raise an error because build_vignettes()
is not exported by the usethis package. Instead users can devtools::build_vignettes()
.
Expected behavior
Building the manual and vignettes.
Session info
─ Session info ──────────────────────────────────────
setting value
version R version 3.6.1 (2019-07-05)
os Ubuntu 18.04.3 LTS
system x86_64, linux-gnu
ui RStudio
language en_US
collate en_US.UTF-8
ctype en_US.UTF-8
tz America/Denver
date 2019-08-27
─ Packages ──────────────────────────────────────────
package * version date lib source
assertthat 0.2.1 2019-03-21 [1] CRAN (R 3.6.0)
backports 1.1.4 2019-04-10 [1] CRAN (R 3.6.0)
bbsAssistant * 1.0.0 2019-08-27 [1] local
callr 3.3.1 2019-07-18 [1] CRAN (R 3.6.1)
cli 1.1.0 2019-03-19 [1] CRAN (R 3.6.0)
crayon 1.3.4 2017-09-16 [1] CRAN (R 3.6.0)
desc 1.2.0 2018-05-01 [1] CRAN (R 3.6.0)
devtools 2.1.0 2019-07-06 [1] CRAN (R 3.6.1)
digest 0.6.20 2019-07-04 [1] CRAN (R 3.6.1)
evaluate 0.14 2019-05-28 [1] CRAN (R 3.6.0)
fs 1.3.1 2019-05-06 [1] CRAN (R 3.6.0)
glue 1.3.1 2019-03-12 [1] CRAN (R 3.6.0)
htmltools 0.3.6 2017-04-28 [1] CRAN (R 3.6.0)
knitr 1.24 2019-08-08 [1] CRAN (R 3.6.1)
magrittr 1.5 2014-11-22 [1] CRAN (R 3.6.0)
memoise 1.1.0 2017-04-21 [1] CRAN (R 3.6.0)
packrat 0.5.0 2018-11-14 [1] CRAN (R 3.6.0)
pkgbuild 1.0.4 2019-08-05 [1] CRAN (R 3.6.1)
pkgload 1.0.2 2018-10-29 [1] CRAN (R 3.6.0)
prettyunits 1.0.2 2015-07-13 [1] CRAN (R 3.6.0)
processx 3.4.1 2019-07-18 [1] CRAN (R 3.6.1)
ps 1.3.0 2018-12-21 [1] CRAN (R 3.6.0)
R6 2.4.0 2019-02-14 [1] CRAN (R 3.6.0)
Rcpp 1.0.2 2019-07-25 [1] CRAN (R 3.6.1)
remotes 2.1.0 2019-06-24 [1] CRAN (R 3.6.1)
rlang 0.4.0 2019-06-25 [1] CRAN (R 3.6.1)
rmarkdown 1.14 2019-07-12 [1] CRAN (R 3.6.1)
rprojroot 1.3-2 2018-01-03 [1] CRAN (R 3.6.0)
rsconnect 0.8.15 2019-07-22 [1] CRAN (R 3.6.1)
rstudioapi 0.10 2019-03-19 [1] CRAN (R 3.6.0)
sessioninfo 1.1.1 2018-11-05 [1] CRAN (R 3.6.0)
testthat 2.2.1 2019-07-25 [1] CRAN (R 3.6.1)
usethis 1.5.1 2019-07-04 [1] CRAN (R 3.6.1)
withr 2.1.2 2018-03-15 [1] CRAN (R 3.6.0)
xfun 0.9 2019-08-21 [1] CRAN (R 3.6.1)
yaml 2.2.0 2018-07-25 [1] CRAN (R 3.6.0)
[1] /home/max/R/x86_64-pc-linux-gnu-library/3.6
[2] /usr/local/lib/R/site-library
[3] /usr/lib/R/site-library
[4] /usr/lib/R/library
Additional context
I've got a PR in the works to provide a quick fix.
I moved the previous README materials into vignettes/vignette.rmd.
I think we should simplify the README such that show only a few images, screenshots, or code chunk echoes which demonstrate the main functionality: retrieving and munging BBS data and analysis results.
Create a helper function to generate a simple jags model.
Recommended by @jaroyle.
Identify what we need to fix/update/add to submit package to CRAN.
See:
@GabsPalomo would you mind making a checklist of what we need to accomplish to submit the first release to CRAN?
Here is a blog (opinion piece) about submitting to CRAN https://www.r-bloggers.com/submitting-your-first-package-to-cran-my-experience/
Perhaps also @juniperlsimonis has some advice for us?!
For some reason, when we specify bibliography: bib.bib
in the html_vignette YAML, the vignette fails to render. Not sure how to ameliorate this.
have tried the following in the yaml:
.bib fn in quotes ("bib.bib"):
bibliography: "bib.bib"
and specifying as:
bibliography:
- bib.bib
Specifying yaml as bibliography: "bib.bib"
yields error:
Could not find bibliography file: bib.bib
Error running filter /Applications/RStudio.app/Contents/MacOS/pandoc/pandoc-citeproc:
Filter returned error status 1
Specifying yaml as bibliography: bib.bib
yields error
Could not find bibliography file: bib.bib
Error running filter /Applications/RStudio.app/Contents/MacOS/pandoc/pandoc-citeproc:
Filter returned error status 1
Error: pandoc document conversion failed with error 83
Specifying yaml as
bibliography:
- bib.bib
yields error:
Could not find bibliography file: bib.bib
Error running filter /Applications/RStudio.app/Contents/MacOS/pandoc/pandoc-citeproc:
Filter returned error status 1
Error: pandoc document conversion failed with error 83
This is recommendation based on my experience while reviewing this package for JOSS (openjournals/joss-reviews#1768).
Is your feature request related to a problem? Please describe.
I was confused by the API for subset_speciesList
. My reading of the vignette was that passing aou.ind
would subset the data to only the provided AOU. This matched my intuition for how a function named subset_speciesList
would work. However, as documented in the function documentation the behavior is actually the opposite, with the list of species provided being removed and all other species kept.
Describe the solution you'd like
I would consider either renaming this function to something like remove_species
or changing the argument names to something like drop_species
. This would make the i
Describe alternatives you've considered
Clarifying the vignette to make the behavior clearer would be another option, but I think intuitive naming would go a long way to preventing user confusion.
See nicholasjclark/BBS.occurrences for potential overlap and added functionality.
Point to this in our documentation as an additional resource.
Bird names in get_credibility_trends function are getting cut off. Need to figure out why and fix it.
The current function get_credibility_trends does not explicitly allow for urls retrieved for individual species, rather it works for "region" specific estimates (which lists all species).
See the 'whatsnew.txt' for the 2019 data update here.
Our function, get_regions() relies on the README.txt which lists the region .zip filenames.
Region filenames will now be located at file
So, TO DO =
GRRRRR
devtools::install_github("trashbirdecology/bbsAssistant", force=TRUE, build_vignettes = TRUE)
devtools::install_github("trashbirdecology/bbsAssistant", force=TRUE, build_vignettes = TRUE)
Downloading GitHub repo trashbirdecology/bbsAssistant@master
✔ checking for file ‘/private/var/folders/f9/_41zf4tn4dz6bzvqyrhpp1fjs58h7x/T/Rtmp1bBXTq/remotes6f332c1174dc/TrashBirdEcology-bbsAssistant-e3ed4d7/DESCRIPTION’ ...
─ preparing ‘bbsAssistant’:
✔ checking DESCRIPTION meta-information ...
─ installing the package to build vignettes
E creating vignettes (10s)
--- re-building ‘quick-overview.Rmd’ using rmarkdown
Skipping install of 'bbsAssistant' from a github remote, the SHA1 (e3ed4d7) has not changed since last install.
Useforce = TRUE
to force installation
TERMS OF USE: North American Breeding Bird Survey Data:
Users of these BBS data are obligated to formally recognize their use of the program's
data in publications, presentations and other outlets. Additionally, all work using these data
should acknowledge the thousands of U.S. and Canadian participants who annually perform
and coordinate the survey. It is in the best interest for the continued success of
the BBS that authors submit a reprint or pdf of their work featuring BBS data
to the National BBS staff for inclusion in the program's bibliography.
If a publication is based solely on the analysis of BBS data, we recommend
that you involve National BBS staff with the writing and/or review of the manuscript.
DATA LIABILITY DISCLAIMER: North American Breeding Bird Survey DataThis database, identified as the North American Breeding Bird Survey Dataset,has been approved for release and publication by the U.S. Geological Survey (USGS)and the Canadian Wildlife Service of Environment Canada (EC). Although this databasehas been subjected to rigorous review and is substantially complete, the USGS and ECreserve the right to revise the data pursuant to further analysis and review.Furthermore, it is released on the condition that the USGS, the U.S. Government,the EC, and the Canadian Government may not be held liable for any damages resulting from its authorized or unauthorized use.
Warning in dir.create(bbsDir) :
'/private/var/folders/f9/_41zf4tn4dz6bzvqyrhpp1fjs58h7x/T/RtmpEHusTt/Rbuild6f6e28359967/bbsAssistant/raw-data' already exists
Loading required package: dplyr
Attaching package: 'dplyr'
The following objects are masked from 'package:stats':
filter, lag
The following objects are masked from 'package:base':
intersect, setdiff, setequal, union
Quitting from lines 49-54 (quick-overview.Rmd)
Error: processing vignette 'quick-overview.Rmd' failed with diagnostics:
No files to import. If state.names
is specified, please ensure files exist in data.dir
.
--- failed re-building ‘quick-overview.Rmd’
SUMMARY: processing the following file failed:
‘quick-overview.Rmd’
Error: Vignette re-building failed.
Execution halted
Error: Failed to install 'bbsAssistant' from GitHub:
System command error, exit status: 1, stdout + stderr (last 10 lines):
E> Quitting from lines 49-54 (quick-overview.Rmd)
E> Error: processing vignette 'quick-overview.Rmd' failed with diagnostics:
E> No files to import. If state.names
is specified, please ensure files exist in data.dir
.
E> --- failed re-building ‘quick-overview.Rmd’
E>
E> SUMMARY: processing the following file failed:
E> ‘quick-overview.Rmd’
E>
E> Error: Vignette re-building failed.
E> Execution halted
Add functionality for BBS strata-level downloading, munging, etc.
To functions:
-[ ] quick_get_data()
-[ ] download_bbs()
Create README.md from README.rmd after getting bulk of functions imported and cleaned.
Add a quick runthrough of primary features to the readme.rmd
spp.lists <- get_species(data.dir=bbsDir)
downloaded 45 KB
Warning in file.rename(from = aou.unzip.path, to = paste0(data.dir, "aou_alpha.txt")) :
cannot rename file '/Users/cchizinski2/Documents/git/bbsAssistant/raw-data/list19p.csv' to '/Users/cchizinski2/Documents/git/bbsAssistant/raw-data/aou_alpha.txt', reason 'No such file or directory'
Warning in file(file, "rt") :
cannot open file '/Users/cchizinski2/Documents/git/bbsAssistant/raw-data/aou_alpha.txt': No such file or directory
Error in file(file, "rt") : cannot open the connection
sessionInfo()
R version 3.6.1 (2019-07-05)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Mojave 10.14.6
Matrix products: default
BLAS: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.6/Resources/lib/libRlapack.dylib
locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8
attached base packages:
[1] stats graphics grDevices utils datasets methods base
other attached packages:
[1] dplyr_0.8.3 bbsAssistant_1.0.0
loaded via a namespace (and not attached):
[1] Rcpp_1.0.3 pillar_1.4.2 compiler_3.6.1 prettyunits_1.0.2 remotes_2.1.0 tools_3.6.1 testthat_2.3.0 digest_0.6.23 pkgbuild_1.0.6
[10] pkgload_1.0.2 tibble_2.1.3 memoise_1.1.0 evaluate_0.14 pkgconfig_2.0.3 rlang_0.4.2 cli_1.1.0 rstudioapi_0.10 curl_4.2
[19] yaml_2.2.0 xfun_0.11 withr_2.1.2 knitr_1.26 desc_1.2.0 fs_1.3.1 devtools_2.2.1 tidyselect_0.2.5 rprojroot_1.3-2
[28] glue_1.3.1 here_0.1 R6_2.4.1 processx_3.4.1 rmarkdown_1.16 sessioninfo_1.1.1 purrr_0.3.3 callr_3.3.2 magrittr_1.5
[37] backports_1.1.5 ps_1.3.0 ellipsis_0.3.0 htmltools_0.4.0 usethis_1.5.1 assertthat_0.2.1 crayon_1.3.4
This is recommendation based on my experience while reviewing this package for JOSS (openjournals/joss-reviews#1768).
Is your feature request related to a problem? Please describe.
Downloading data for one or more specific states as described in the vignette takes a number of steps:
which
and stringr::str_detect
to find the base filenameget_bbsData
Since this is a core functionality for the package it would be nice to hide this complexity from the user.
Describe the solution you'd like
Move the first three steps inside of get_bbsData
and have the get_bbsData
argument be region
so that a call to get the data for Florida looks like: get_bbsData(region = 'Florida')
.
citation("bbsAssistant")
yields the following error:
Error in .bibentry_check_bibentry1(rval) :
A bibentry of bibtype ‘Article’ has to specify the fields: author, title, journal, year
Need to fix the relevant .bib
Describe the bug
It appears that using overwrite.bbs = TRUE
does not allow `download_bbs() to download the data
suppressMessages(download_bbs(country.names="USA", data.dir = bbsDir, overwrite.bbs = TRUE, overwrite.routes = TRUE))
[1] "Data already exists. No data downloaded. To force installation, specify overwrite.bbs=TRUE
, or delete local files."
[1] "Data already exists. No data downloaded. To force installation, specify overwrite.bbs=TRUE
, or delete local files."
sessionInfo()
R version 3.6.1 (2019-07-05)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Mojave 10.14.6
Matrix products: default
BLAS: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.6/Resources/lib/libRlapack.dylib
locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8
attached base packages:
[1] stats graphics grDevices utils datasets methods base
other attached packages:
[1] bbsAssistant_1.0.0
loaded via a namespace (and not attached):
[1] Rcpp_1.0.3 pillar_1.4.2 compiler_3.6.1 prettyunits_1.0.2 remotes_2.1.0 tools_3.6.1 testthat_2.3.0 digest_0.6.23 pkgbuild_1.0.6
[10] pkgload_1.0.2 tibble_2.1.3 memoise_1.1.0 evaluate_0.14 pkgconfig_2.0.3 rlang_0.4.2 cli_1.1.0 rstudioapi_0.10 curl_4.2
[19] yaml_2.2.0 xfun_0.11 withr_2.1.2 dplyr_0.8.3 knitr_1.26 desc_1.2.0 fs_1.3.1 devtools_2.2.1 tidyselect_0.2.5
[28] rprojroot_1.3-2 glue_1.3.1 here_0.1 R6_2.4.1 processx_3.4.1 rmarkdown_1.16 sessioninfo_1.1.1 purrr_0.3.3 callr_3.3.2
[37] magrittr_1.5 backports_1.1.5 ps_1.3.0 ellipsis_0.3.0 htmltools_0.4.0 usethis_1.5.1 assertthat_0.2.1 crayon_1.3.4
For each function we need to go through and improve the:
And add @examples to the functions
We need to write a test for function get_credibility_trends.
Request a review of package functionality and potential issues from Pardieck et al.
Add testing
The column names for the object outputted from the functions get_credibility_trends + get_analysis_results should be the same.
Not sure if this should be an issue, but I was wondering what type of information you want to include in the summary.
I understand that the package is basically downloading the data that already exists in this website to make it more user friendly and more importantly R-friendly. However, would you like to include the following in this summary?
These are the only things I can think of at the moment to include in the summary. If you agree with them I can write them up for your review.
Figure out how to fix the manual to show the authors/maintainer etc.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.