Skip to content

Added GeoCharts to visualize where the volunteers are from#309

Open
SayanDey322 wants to merge 2 commits into
pyladies:mainfrom
SayanDey322:develop
Open

Added GeoCharts to visualize where the volunteers are from#309
SayanDey322 wants to merge 2 commits into
pyladies:mainfrom
SayanDey322:develop

Conversation

@SayanDey322
Copy link
Copy Markdown

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 19, 2026

Deploy Preview for pyladiescon-portal-docs ready!

Name Link
🔨 Latest commit 9fdb451
🔍 Latest deploy log https://app.netlify.com/projects/pyladiescon-portal-docs/deploys/6a03643b6ae66f0008834b5a
😎 Deploy Preview https://deploy-preview-309--pyladiescon-portal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@SayanDey322
Copy link
Copy Markdown
Author

Added GeoCharts to visualize where the volunteers are from

Comment thread portal/common.py
result = [[data["region"], data["count"]] for data in volunteers_by_region]
# Map continents to representative countries for GeoChart
continent_to_country = {
"North America": "United States",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is incorrect representation of the data.
I understand this is a limitation of the geocharts library but because there are other countries in north america that is not just united states, the chart now shows wrong information, and not what we want.

Copy link
Copy Markdown
Member

@Mariatta Mariatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the comment. The chart replaces region data with countries like United States and Brazil and is not a real representation of our volunteers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants