Skip to content

Update cities.csv#71

Closed
goldhaxx wants to merge 1 commit intoda-luce:mainfrom
goldhaxx:patch-1
Closed

Update cities.csv#71
goldhaxx wants to merge 1 commit intoda-luce:mainfrom
goldhaxx:patch-1

Conversation

@goldhaxx
Copy link

@goldhaxx goldhaxx commented Feb 7, 2025

Added Salt Lake City, UT to cities.csv

Added Salt Lake City, UT to cities.csv
@da-luce
Copy link
Owner

da-luce commented Feb 13, 2025

The cities.csv file is generated using a script that filters these cities with a set population cutoff. I agree that Salt Lake City should probably be included, so if you want to regenerate the file with a population threshold of 200,000, I'd be happy to accept this PR.

@goldhaxx
Copy link
Author

@da-luce how was cities15000.txt generated?

@da-luce
Copy link
Owner

da-luce commented Mar 1, 2025

Sorry for leaving that out. GeoNames provides a zip to the file.

@da-luce
Copy link
Owner

da-luce commented Aug 29, 2025

Since cities.csv now has a population threshold of 200,000 (see filter_cities.py), Salt Lake City is now included. I confirmed this works with: astroterm -i "Salt Lake City"

@da-luce da-luce closed this Aug 29, 2025
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