Skip to content
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

Add An Arbitrary World and An Arbitrary World Lite maps #46

Merged
merged 3 commits into from
Feb 7, 2025

Conversation

TheNathanSpace
Copy link
Contributor

@TheNathanSpace TheNathanSpace commented Feb 4, 2025

One of the most popular GeoGuessr community maps is An Arbitrary World (AAW): https://www.geoguessr.com/maps/6089bfcff6a0770001f645dd. This PR adds two maps that are based on AAW!

AAW is built using an algorithm to generate a list of locations evenly distributed across the world, by country, using official vehicle coverage. You can use the same algorithm by using the creator's tool here: https://github.com/slashP/Vali, which is what I did to make these maps.

I tried a few different map sizes, and settled on the location quantities of 5000 for the main version and 2500 for the Lite version. On my computer, here are their load times:

  • 2500: 3 seconds
  • 5000: 15 seconds
  • 10000: 100 seconds (Not included in the PR, but included here to demonstrate why I chose these location quantities.)

The maps are FeatureCollections of Point Features. If you know of any way to optimize map loading on my side, it would be great to use larger maps instead—but if you don't, these are the most reasonable sizes to me.

The images are sourced from Unsplash, edited (e.g., to size), and uploaded to Imgur.

Note that I HAVEN'T changed maps.json—I've instead changed list.maps.json. It looks like the CI pipelines will automatically update maps.json?

I've tried to follow all the instructions you have in this repository and the instructions I've found in past map PRs you've accepted, but I'm happy to adjust anything you want—just let me know!

@TheNathanSpace
Copy link
Contributor Author

Hey @BilelJegham, let me know if you have any thoughts on this! :) (no pressure if you're busy though)

@BilelJegham BilelJegham merged commit 2157cc8 into GeoGuess:main Feb 7, 2025
2 checks passed
@BilelJegham
Copy link
Member

All good :)

Thanks @TheNathanSpace for the contribution

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