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

make admins & timezones required #4467

Open
nilsnolde opened this issue Dec 20, 2023 · 10 comments · May be fixed by #4469
Open

make admins & timezones required #4467

nilsnolde opened this issue Dec 20, 2023 · 10 comments · May be fixed by #4469
Labels

Comments

@nilsnolde
Copy link
Member

we talked about it and came to the conclusion that there's no point to have timezones & admins optional. there's tons of issues with no admins added, and also having no timezones results in strange behavior if someone requests with time. @kevinkreiser suggested we could host those somewhere so people can pull them without building. we could use github artifacts for that, triggered by a manual github action to re-create them. those artifacts are available for download via the github api. though they're only persisted for 90 days or so I think, but it's easy to automatically generate them every couple of months, github actions has cron capabilities.

could even auto-download them in the code eventually via a config url defaulting to the latest github artifact, but the whole HTTP/curl compilation logic seems a bit daunting currently..

@kevinkreiser
Copy link
Member

i was thinking of having them be part of a "release" so that they persist forever. it is often the case for admins that you basically dont want to touch them if you finally found a version of the planet where all of them seem unbroken 😄 for timezones it changes infrequently so the same applies but for a different reason

@kevinkreiser
Copy link
Member

this here where we can attach binaries to a release:
image

@nilsnolde
Copy link
Member Author

Ah also cool actually. Then we’d ideally release every time we know admin stuff or timezone stuff changed (not counting broken osm shit of course). that’ll be on a tighter schedule than our current release habits;)

@kevinkreiser
Copy link
Member

hahahaha sick burn 😄

@nilsnolde nilsnolde linked a pull request Dec 21, 2023 that will close this issue
@ianthetechie
Copy link
Contributor

it is often the case for admins that you basically dont want to touch them if you finally found a version of the planet where all of them seem unbroken

Yeah, I'm with @kevinkreiser on this one ;) in practice I update admins maybe once/month tops and have never had any issues / heard complaints from end users.

Given that fact, I have it on my TODO list to look at using Daylight for admins. I'll report back what I find (someone poke me please if I don't update within a few weeks), but it seems rather appealing for this use case, as it 1) will be reasonably up to date, 2) is completely OSM data, but 3) ostensibly tends to be "less broken" on average.

@nilsnolde
Copy link
Member Author

Don’t we even parse ALL of the admins, despite only using level 2 & 4? I’ve never seen a broken country, and I check regularly. States maybe in some places? Even if, that’s hardly relevant. In other words, I’d not be too worried about broken admins either for Valhalla. For geocoding/other purposes that’s a totally different discussion.

@ianthetechie
Copy link
Contributor

Don’t we even parse ALL of the admins, despite only using level 2 & 4?

Quite possibly...

I’ve never seen a broken country, and I check regularly. States maybe in some places? Even if, that’s hardly relevant. In other words, I’d not be too worried about broken admins either for Valhalla.

I've never had anyone complain with our hosted services either, but that may be a function of it being heavily skewed towards NA and EU which have pretty good data and large communities.

The main reason I think Daylight / Overture are interesting is that it's purely OSM derived, on a rolling snapshot + patches basis (and is also fairly certain to have any vandalism of admin names reverted, though then again that has also been fairly rare; it's more commonly an issue with city and other place names). Those are interesting properties (IMO) for published "base" datasets from the Valhalla project. Just a thought.

@nilsnolde
Copy link
Member Author

Yeah sorry, I should’ve made it clearer that this is my personal, heavily biased opinion against those distros. You’re very likely right regarding admins in Daylight. In fact I think it’s the only thing it’s useful for in real life.

@kevinkreiser
Copy link
Member

kevinkreiser commented Dec 24, 2023

it's also very useful if you're worried about buildings coverage in your map display since it has a bunch of added sources for that

@nilsnolde
Copy link
Member Author

aren't the additional footprints AI generated? in my book that'd be barely usable. seems they add some ESRI data, but sounds like only for addressing, not footprints. admittedly their footprint source description is just 2 sentences.

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 a pull request may close this issue.

3 participants