ropensci / software-review

rOpenSci Software Peer Review.
286 stars 104 forks source link

feowR: Download shapefiles of Freshwater Ecoregions of the World (FEOW) #647

Open brunomioto opened 1 week ago

brunomioto commented 1 week ago

Submitting Author Name: Bruno Henrique Mioto Stabile Submitting Author Github Handle: !--author1-->@brunomioto<!--end-author1-- Other Package Authors Github handles: (comma separated, delete if none) Repository: https://github.com/brunomioto/feowR Version submitted: Submission type: Standard Editor: TBD Reviewers: TBD

Archive: TBD Version accepted: TBD Language: en


Package: feowR
Title: Download shapefiles of Freshwater Ecoregions of the World (FEOW)
Description: Get access to all shapefiles from Freshwater Ecoregions of the World (FEOW) and a list with IDs and names.
Version: 0.0.0.9000
Authors@R: 
    c(person(given = "Bruno",
             family = "Mioto",
             role = c("aut", "cre"),
             email = "brunomioto97@gmail.com"),
      person(given = "The Nature Conservancy and World Wildlife Fund, Inc",
             role = "cph"))
License: CC BY 4.0 + file LICENSE
Encoding: UTF-8
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.3.1
URL: https://github.com/brunomioto/feowR, https://brunomioto.github.io/feowR/
BugReports: https://github.com/brunomioto/feowR/issues
Imports: 
    cli,
    curl,
    piggyback,
    sf
Depends: 
    R (>= 2.10)
LazyData: true
Suggests: 
    dplyr,
    ggplot2,
    testthat (>= 3.0.0)
Config/testthat/edition: 3
Config/Needs/website: rmarkdown

Scope

It helps the user to retrieve shapefiles data from Freshwater Ecoregions of the World (FEOW) and save it on the working directory.

Main use is for biologists (like me) who needs to use this data in other analyses, like selecting points that falls inside each ecoregion.

No

Technical checks

Confirm each of the following by checking the box.

This package:

Publication options

MEE Options - [ ] The package is novel and will be of interest to the broad readership of the journal. - [ ] The manuscript describing the package is no longer than 3000 words. - [ ] You intend to archive the code for the package in a long-term repository which meets the requirements of the journal (see [MEE's Policy on Publishing Code](http://besjournals.onlinelibrary.wiley.com/hub/journal/10.1111/(ISSN)2041-210X/journal-resources/policy-on-publishing-code.html)) - (*Scope: Do consider MEE's [Aims and Scope](http://besjournals.onlinelibrary.wiley.com/hub/journal/10.1111/(ISSN)2041-210X/aims-and-scope/read-full-aims-and-scope.html) for your manuscript. We make no guarantee that your manuscript will be within MEE scope.*) - (*Although not required, we strongly recommend having a full manuscript prepared when you submit here.*) - (*Please do not submit your package separately to Methods in Ecology and Evolution*)

Code of conduct

ropensci-review-bot commented 1 week ago

Thanks for submitting to rOpenSci, our editors and @ropensci-review-bot will reply soon. Type @ropensci-review-bot help for help.

ropensci-review-bot commented 1 week ago

:rocket:

Editor check started

:wave:

mpadge commented 6 days ago

Sorry that the checks haven't appeared @brunomioto. GitHub has recently been responding to some of the necessary steps with "Unexpected error" messages. Your glitch has provided enough information for us to fix on our side so it shouldn't happen again. In the meantime, I'll manually push the checks in a moment.

ropensci-review-bot commented 6 days ago

Checks for feowR (v0.0.0.9000)

git hash: 90d82190

Package License: CC BY 4.0 + file LICENSE


1. Package Dependencies

Details of Package Dependency Usage (click to open)

The table below tallies all function calls to all packages ('ncalls'), both internal (r-base + recommended, along with the package itself), and external (imported and suggested packages). 'NA' values indicate packages to which no identified calls to R functions could be found. Note that these results are generated by an automated code-tagging system which may not be entirely accurate. |type |package | ncalls| |:----------|:---------|------:| |imports |cli | NA| |imports |httr | NA| |imports |piggyback | NA| |imports |sf | NA| |suggests |dplyr | NA| |suggests |ggplot2 | NA| |suggests |testthat | NA| |linking_to |NA | NA| Click below for tallies of functions used in each package. Locations of each call within this package may be generated locally by running 's <- pkgstats::pkgstats()', and examining the 'external_calls' table. **NOTE:** No imported packages appear to have associated function calls; please ensure with author that these 'Imports' are listed appropriately.


2. Statistical Properties

This package features some noteworthy statistical properties which may need to be clarified by a handling editor prior to progressing.

Details of statistical properties (click to open)

The package has: - code in R (100% in 2 files) and - 1 authors - 1 vignette - 1 internal data file - 4 imported packages - 1 exported function (median 19 lines of code) - no non-exported function in R (median lines of code) --- Statistical properties of package structure as distributional percentiles in relation to all current CRAN packages The following terminology is used: - `loc` = "Lines of Code" - `fn` = "function" - `exp`/`not_exp` = exported / not exported All parameters are explained as tooltips in the locally-rendered HTML version of this report generated by [the `checks_to_markdown()` function](https://docs.ropensci.org/pkgcheck/reference/checks_to_markdown.html) The final measure (`fn_call_network_size`) is the total number of calls between functions (in R), or more abstract relationships between code objects in other languages. Values are flagged as "noteworthy" when they lie in the upper or lower 5th percentile. |measure | value| percentile|noteworthy | |:------------------------|-----:|----------:|:----------| |files_R | 2| 13.6| | |files_vignettes | 1| 68.4| | |files_tests | 3| 75.2| | |loc_R | 21| 1.7|TRUE | |loc_vignettes | 31| 4.2|TRUE | |loc_tests | 24| 14.8| | |num_vignettes | 1| 64.8| | |data_size_total | 5417| 67.6| | |data_size_median | 5417| 74.9| | |n_fns_r | 1| 0.0|TRUE | |n_fns_r_exported | 1| 0.0|TRUE | |n_fns_r_not_exported | 0| 0.0|TRUE | |n_fns_per_file_r | 1| 0.2|TRUE | |num_params_per_fn | 1| 1.6|TRUE | |loc_per_fn_r | 19| 57.3| | |loc_per_fn_r_exp | 19| 44.7| | |rel_whitespace_R | 62| 11.6| | |rel_whitespace_vignettes | 52| 10.5| | |rel_whitespace_tests | 75| 33.0| | |doclines_per_fn_exp | 19| 12.2| | |fn_call_network_size | 0| 0.0|TRUE | ---

2a. Network visualisation

Click to see the interactive network visualisation of calls between objects in package


3. goodpractice and other checks

Details of goodpractice checks (click to open)

#### 3a. Continuous Integration Badges [![R-CMD-check.yaml](https://github.com/brunomioto/feowR/actions/workflows/R-CMD-check.yaml/badge.svg)](https://github.com/brunomioto/feowR/actions) **GitHub Workflow Results** | id|name |conclusion |sha | run_number|date | |----------:|:--------------------------|:----------|:------|----------:|:----------| | 9637299025|pages build and deployment |success |ea0505 | 15|2024-06-23 | | 9637288502|pkgcheck |success |90d821 | 3|2024-06-23 | | 9637288498|pkgdown |success |90d821 | 14|2024-06-23 | | 9637288499|R-CMD-check |success |90d821 | 7|2024-06-23 | | 9637288503|test-coverage |success |90d821 | 10|2024-06-23 | --- #### 3b. `goodpractice` results #### `R CMD check` with [rcmdcheck](https://r-lib.github.io/rcmdcheck/) rcmdcheck found no errors, warnings, or notes #### Test coverage with [covr](https://covr.r-lib.org/) Package coverage: 94.12 #### Cyclocomplexity with [cyclocomp](https://github.com/MangoTheCat/cyclocomp) No functions have cyclocomplexity >= 15 #### Static code analyses with [lintr](https://github.com/jimhester/lintr) [lintr](https://github.com/jimhester/lintr) found the following 5 potential issues: message | number of times --- | --- Avoid library() and require() calls in packages | 3 Lines should not be more than 80 characters. This line is 93 characters. | 1 Lines should not be more than 80 characters. This line is 94 characters. | 1


Package Versions

|package |version | |:--------|:--------| |pkgstats |0.1.5.2 | |pkgcheck |0.1.2.42 |


Editor-in-Chief Instructions:

This package is in top shape and may be passed on to a handling editor

brunomioto commented 6 days ago

Just updated the package adding a test for "No internet connection". The code is now 100% covered.

jooolia commented 1 day ago

Dear @brunomioto, Thank you for your submission. We are in the process of looking for a handling editor. Thanks, Julia