student: round out the Kyle Walker mapping toolkit (mapgl, crsuggest, idbr)#15
Merged
Conversation
… idbr) mapgl gives WebGL vector maps (MapLibre) with no token via maplibre() + CARTO styles — the NYT look as an htmlwidget that publishes to GitHub Pages. crsuggest answers the CRS-choice question in one function. idbr adds Census International Data Base access using the same Census API key students already need for tidycensus. mapboxapi deliberately excluded: it would require a per-student Mapbox account/token. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR expands the student devcontainer image’s R mapping/census capabilities by adding the remaining course-relevant Kyle Walker ecosystem packages and updating documentation to match. It complements the existing sf/tidycensus setup and extends the Dockerfile smoke test to ensure the full mapping toolkit loads successfully.
Changes:
- Install additional R packages in the
studentimage:mapgl,crsuggest, andidbr(alongsidesfandtidycensus). - Extend the Docker build smoke test to load all mapping-related packages (
sf,tidycensus,tigris,mapgl,crsuggest,idbr). - Update README and CLAUDE documentation to describe the expanded mapping toolkit and the intentional exclusion of
mapboxapi.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
student/Dockerfile |
Installs mapgl, crsuggest, idbr and expands the mapping smoke test package-load list. |
README.md |
Updates the student image feature list to include the added Kyle Walker mapping packages. |
CLAUDE.md |
Updates the student/ image documentation to reflect the expanded mapping/census stack and rationale. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds the remaining course-relevant Kyle Walker packages to the student image:
mapgl— WebGL vector maps (MapLibre/Mapbox GL htmlwidgets). Token-free viamaplibre()+ CARTO styles; publishes to GitHub Pages like the other htmlwidgets.crsuggest— suggests the right projected CRS for a dataset.idbr— Census International Data Base; uses the same Census API key students already need for tidycensus.mapboxapideliberately excluded (per-student Mapbox account/token required). Smoke test extended to load all six mapping packages.Validation build on this branch: green across base/dev/student — https://github.com/PPBDS/devcontainers/actions/runs/28608205412
🤖 Generated with Claude Code