Skip to content

Publish /en/lessons/choropleth-maps-python-folium #3551

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: gh-pages
Choose a base branch
from

Conversation

anisa-hawes
Copy link
Contributor

@anisa-hawes anisa-hawes commented May 2, 2025

Preparing files for publication.

Closes #3550

Checklist

  • Assign yourself in the "Assignees" menu
  • Add the appropriate "Label"
  • If this PR closes an Issue, add the phrase Closes #ISSUENUMBER to your summary above
  • Ensure the status checks pass: if you have difficulty fixing build errors, please contact our Publishing Manager @anisa-hawes
  • Check the Netlify Preview: navigate to netlify/ph-preview/deploy-preview and click 'details' (at right)
  • Assign at least one individual or team to "Reviewers"
    • [ ] if the text needs to be translated, please follow the translation request guidelines, then assign the relevant language team(s) as "Reviewers" and tag both the team as well as the managing editor in your PR.

Add bio for Adam Porter
Create Markdown file for new EN original lesson.
@anisa-hawes anisa-hawes self-assigned this May 2, 2025
@anisa-hawes
Copy link
Contributor Author

Hello Alex @hawc2,

I’ve staged choropleth-maps-python-folium for publication. You can read it through in Preview and let me know if you note anything which you'd like to adjust.

We can publish this next week!

Thank you.

@anisa-hawes anisa-hawes requested a review from hawc2 May 2, 2025 14:04
@hawc2
Copy link
Contributor

hawc2 commented May 5, 2025

@anisa-hawes Looks good. A couple minor things:

This sentence - "According to Dombrowski, Gniady, and Kloster, Jupyter Notebook is" - it should be "Jupyter Notebooks are"

There's a line of code in the Getting County Geometry Data that I'm not sure I understand - why is this code line commented out? What purpose does it serve to include it at all?:
"# counties = gpd.read_file("https://www2.census.gov/geo/tiger/GENZ2021/shp/cb_2021_us_county_5m.zip")"

The paragraph before seems to suggest either code would work, it's a bit confusing.

That's all I see, it otherwise looks great to me and ready to go!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preparing to publish a new EN original lesson: choropleth-maps-python-folium
2 participants