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 alt property validation #2
Comments
Is this library currently in-use? I was looking over the |
This repo is still useful in finding out the root cause of a failed dist build, for example.. from the README:
|
@missinglink The goal is to replace the core/common tooling that was initially written in Python with Go versions. The goal is ensure that these tools can a) be free of the kinds of dependency challenges that Python can introduce b) even if distributed as platform-specific binary tools. It's slow progress, for all the usual reasons, but we'll get there eventually. |
Gotcha, the reason I asked is I wasn't sure if we should be making changes like this in the |
As a practical matter, the first place they should be made is in the Python code since that's "stable" and what @stepps00 uses for most of his work. The gotcha there is that the Best practice should be to ensure that both the Python and Go code get updated at the same, unless dealing with the Go version involves major rewrites or refactoring; things like "Oh look, now I need to port Ideally, I'd like for WOF to have "core" tools in both Python and Go. Python is more of a common currency than Go and is more amenable to the kinds of utility scripts and tools that Stephen needs to do his job. Go is better for making tools that we can point people to when they just want to make a quick edit or contribution (and need to "exportify" their changes). It also demonstrates that if it's possible to maintain common tools in those languages, they can be the templates for [ all the other languages ]. |
WOF requires five properties in alt files, but as far as I know, there are no tools to ensure these five properties are present. Properties include:
src:alt_label
src:geom
wof:geomhash
wof:id
wof:repo
Opening this issue to track a validation method to check for all of these required properties, assuming there isn't one already.
The text was updated successfully, but these errors were encountered: