Skip to content

Commit

Permalink
Implement usmapdata 0.2.2 updates (#93)
Browse files Browse the repository at this point in the history
* Use `usmapdata 0.2.2` functions in `usmap_transform()`
* Removed references to `as_sf` parameter in `usmapdata` function calls.
* Updated GitHub Actions workflows to resolve deprecation warnings (see
related: [pdil/usmapdata/23](pdil/usmapdata#27))
  • Loading branch information
pdil authored Mar 10, 2024
2 parents f5c81cc + aeb77b7 commit 9a375e2
Show file tree
Hide file tree
Showing 9 changed files with 36 additions and 60 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Workflow derived from https://github.com/r-lib/actions/tree/v2/examples
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help

on:
push:
branches: [main, master]
Expand Down Expand Up @@ -29,7 +29,7 @@ jobs:
R_KEEP_PKG_SOURCE: yes

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- uses: r-lib/actions/setup-pandoc@v2

Expand All @@ -46,4 +46,5 @@ jobs:

- uses: r-lib/actions/check-r-package@v2
with:
upload-snapshots: true
upload-snapshots: true
build_args: 'c("--no-manual","--compact-vignettes=gs+qpdf")'
8 changes: 2 additions & 6 deletions .github/workflows/lintr.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,3 @@
# This workflow uses actions that are not certified by GitHub.
# They are provided by a third-party and are governed by
# separate terms of service, privacy policy, and support
# documentation.
# lintr provides static code analysis for R.
# It checks for adherence to a given style,
# identifying syntax errors and possible semantic issues,
Expand Down Expand Up @@ -29,7 +25,7 @@ jobs:

steps:
- name: Checkout code
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Setup R
uses: r-lib/actions/setup-r@v2
Expand All @@ -45,7 +41,7 @@ jobs:
continue-on-error: true

- name: Upload analysis results to GitHub
uses: github/codeql-action/upload-sarif@v2
uses: github/codeql-action/upload-sarif@v3
with:
sarif_file: lintr-results.sarif
wait-for-processing: true
12 changes: 6 additions & 6 deletions .github/workflows/test-coverage.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Workflow derived from https://github.com/r-lib/actions/tree/v2/examples
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help

on:
push:
branches: [main, master]
Expand All @@ -15,7 +15,7 @@ jobs:
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- uses: r-lib/actions/setup-r@v2
with:
Expand All @@ -31,20 +31,20 @@ jobs:
covr::codecov(
quiet = FALSE,
clean = FALSE,
install_path = file.path(Sys.getenv("RUNNER_TEMP"), "package")
install_path = file.path(normalizePath(Sys.getenv("RUNNER_TEMP"), winslash = "/"), "package")
)
shell: Rscript {0}

- name: Show testthat output
if: always()
run: |
## --------------------------------------------------------------------
find ${{ runner.temp }}/package -name 'testthat.Rout*' -exec cat '{}' \; || true
find '${{ runner.temp }}/package' -name 'testthat.Rout*' -exec cat '{}' \; || true
shell: bash

- name: Upload test results
if: failure()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage-test-failures
path: ${{ runner.temp }}/package
path: ${{ runner.temp }}/package
4 changes: 2 additions & 2 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Description: Obtain United States map data frames of varying region types (e.g.
state). The map data frames include Alaska and Hawaii conveniently placed to the
bottom left, as they appear in most maps of the US. Convenience functions for plotting
choropleths, visualizing spatial data, and working with FIPS codes are also provided.
Authors@R: person("Paolo", "Di Lorenzo", email = "[email protected]", role = c("aut", "cre"))
Authors@R: person("Paolo", "Di Lorenzo", email = "[email protected]", role = c("aut", "cph", "cre"))
Depends:
R (>= 3.5.0)
License: GPL (>= 3)
Expand All @@ -16,7 +16,7 @@ URL: https://usmap.dev
BugReports: https://github.com/pdil/usmap/issues
Imports:
rlang,
usmapdata (>= 0.2.0)
usmapdata (>= 0.2.2)
Suggests:
covr,
ggplot2 (>= 3.5.0),
Expand Down
11 changes: 10 additions & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,20 @@

### Improvements
* Add citation information to README, see [Issue #86](https://github.com/pdil/usmap/issues/86).
* Update map theme to work with new legend behavior in [ggplot2 v3.5.0](https://www.tidyverse.org/blog/2024/02/ggplot2-3-5-0-legends/#placement).
* Update map theme to work with new legend behavior in [ggplot2 3.5.0](https://www.tidyverse.org/blog/2024/02/ggplot2-3-5-0-legends/#placement).'

### Bug Fixes
* Replace `size` with `linewidth` in `plot_usmap()` documentation, see [Issue #89](https://github.com/pdil/usmap/issues/89).

### Technical Changes
* Internal `usmapdata` functions are used for data transformation (i.e. `usmap_transform()`) values for consistency.
* This allows the same values used to create the map to be used when transforming external data.
* Values will now only have to be updated in one place.
* `usmapdata 0.2.2` is now a required dependency because of this change.
* Continue `sf`-based map data file migration.
* Usage of the `as_sf` parameter in `usmapdata` function calls has been removed.
* The parameter will be removed from `usmapdata` functions in the future.

# usmap 0.7.0
Released Saturday, January 20, 2024.

Expand Down
10 changes: 5 additions & 5 deletions R/fips.R
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,14 @@
#' @export
fips <- function(state, county = c()) {
if (missing(state) && missing(county)) {
return(usmapdata::fips_data(as_sf = TRUE)$fips)
return(usmapdata::fips_data()$fips)
}

state_ <- tolower(state)
county_ <- tolower(county)

if (length(county_) == 0) {
df <- usmapdata::fips_data(as_sf = TRUE)
df <- usmapdata::fips_data()
abbr <- tolower(df$abbr)
full <- tolower(df$full)
fips2 <- c(df$fips, df$fips)
Expand All @@ -74,7 +74,7 @@ fips <- function(state, county = c()) {
stop("`county` parameter cannot be used with multiple states.")
}

df <- usmapdata::fips_data("counties", as_sf = TRUE)
df <- usmapdata::fips_data("counties")
name <- tolower(df$county)
state_abbr <- tolower(df$abbr)
state_full <- tolower(df$full)
Expand Down Expand Up @@ -174,10 +174,10 @@ fips_info.character <- function(fips, sortAndRemoveDuplicates = FALSE) {
#' @keywords internal
get_fips_info <- function(fips, sortAndRemoveDuplicates) {
if (all(nchar(fips) == 2)) {
df <- usmapdata::fips_data(as_sf = TRUE)
df <- usmapdata::fips_data()
columns <- c("abbr", "fips", "full")
} else if (all(nchar(fips) == 5)) {
df <- usmapdata::fips_data("counties", as_sf = TRUE)
df <- usmapdata::fips_data("counties")
columns <- c("full", "abbr", "county", "fips")
}

Expand Down
2 changes: 1 addition & 1 deletion R/plot-map.R
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ plot_usmap <- function(regions = c("states", "state", "counties", "county"),
if (regions == "state") regions <- "states"
else if (regions == "county") regions <- "counties"

centroid_labels <- usmapdata::centroid_labels(regions, as_sf = TRUE)
centroid_labels <- usmapdata::centroid_labels(regions)

if (length(include) > 0) {
centroid_labels <- centroid_labels[
Expand Down
40 changes: 5 additions & 35 deletions R/transform.R
Original file line number Diff line number Diff line change
Expand Up @@ -118,44 +118,14 @@ perform_transform <- function(data, ...) {
sf::st_agr(transformed) <- "constant"

# Transform Alaska points
ak_bbox <- sf::st_as_sfc(
sf::st_bbox(
c(
xmin = -4377000,
xmax = -1519000,
ymin = 1466000,
ymax = 3914000
),
crs = usmap_crs()
)
)
ak_bbox <- usmapdata:::alaska_bbox()
alaska <- sf::st_intersection(transformed, ak_bbox)

if (nrow(alaska) > 0) {
sf::st_geometry(alaska) <- sf::st_geometry(alaska) * usmapdata:::transform2D(-50, 1 / 2)
sf::st_geometry(alaska) <- sf::st_geometry(alaska) + c(3e5, -2e6)
sf::st_crs(alaska) <- usmap_crs()
}
alaska <- usmapdata:::transform_alaska(alaska)

# Transform Hawaii points
hi_bbox <- sf::st_as_sfc(
sf::st_bbox(
c(
xmin = -5750000,
xmax = -5450000,
ymin = -1050000,
ymax = -441000
),
crs = usmap_crs()
)
)
hi_bbox <- usmapdata:::hawaii_bbox()
hawaii <- sf::st_intersection(transformed, hi_bbox)

if (nrow(hawaii) > 0) {
sf::st_geometry(hawaii) <- sf::st_geometry(hawaii) * usmapdata:::transform2D(-35)
sf::st_geometry(hawaii) <- sf::st_geometry(hawaii) + c(3.6e6, 1.8e6)
sf::st_crs(hawaii) <- usmap_crs()
}
hawaii <- usmapdata:::transform_hawaii(hawaii)

# Re-combine all points
transformed_excl_ak <- sf::st_difference(transformed, ak_bbox)
Expand All @@ -177,5 +147,5 @@ perform_transform <- function(data, ...) {
#'
#' @export
usmap_crs <- function() {
sf::st_crs(9311)
usmapdata:::ea_crs()
}
2 changes: 1 addition & 1 deletion R/usmap.R
Original file line number Diff line number Diff line change
Expand Up @@ -103,5 +103,5 @@
us_map <- function(regions = c("states", "state", "counties", "county"),
include = c(),
exclude = c()) {
usmapdata::us_map(regions = regions, include = include, exclude = exclude, as_sf = TRUE)
usmapdata::us_map(regions = regions, include = include, exclude = exclude)
}

0 comments on commit 9a375e2

Please sign in to comment.