-
Notifications
You must be signed in to change notification settings - Fork 3
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
Handle range labels better #34
Comments
Now that I look at I think there are a few questions to be answered:
|
|
It seems like trouble to specify place names as we're doing it right now. For example: "places": {
"Chesapeake": {
"gnis_id": 1496841
},
"Suffolk": {
"gnis_id": 1500187
},
"Virginia Beach": {
"gnis_id": 1500261
}
} In most programming languages, isn't "Virginia Beach" going to be the name of a variable? Are we really giving people a variable with spaces in it? Explore whether this is going to be trouble. Even if we pretend that people don't care about the phrase "Virginia Beach," since we have a GNIS ID etc. (which I doubt is true), there are places for which we have no such identifier. |
On reflection, I don't think that it's a problem having spaces in these JSON labels. I'm going to play around with a bit more, but I don't think that there's any realistic way in which anybody is manipulating this data for which it's a problem. |
This is an actual excerpt from
crow.json
:national_forest_lands_and_department_lands
? That's meaningless. We should make two changes here.The first is to move to the
bear.json
example, which is to simply number these ranges, rather than try to name them. The groupings are frequently not logical in a way that lends themselves to naming.The second is to figure out how to actually represent a concept like "national forest lands and department lands." Representing "Virginia" is easy (it has a GNIS ID, after all), but there's no ID for all national forest lands and department lands.
The text was updated successfully, but these errors were encountered: