CAM-Gerlach's Comments
Changeset | When | Comment |
---|---|---|
169836414 | 8 days ago | Hey, thanks for adding sidewalks! Quick tip: when you've gone to the work of adding separate sidewalks like this, don't forget to add the `sidewalk:<left/right/both>=separate/no` tags on their respective streets, so they get properly tagged and picked up as such per approved OSM practice as well as accepted local convention. Thanks!
|
169796175 | 9 days ago | Thanks for fixing the issue from changset #169697932 !
|
169697932 | 9 days ago | Flagging as SEO spam per discussion; fixed by Udarian in changeset #169796175
|
169624416 | 11 days ago | Thanks! BTW, just a heads up you're welcome to join the new #local-nrv channel on the OSM-US Slack workspace https://openstreetmap.us/get-involved/slack/ , and/or the Map NRV Discord server https://discord.gg/TYyeyjYh (both created by other prolific local mappers) for a place to discuss things and chat with the local mapping community, in case you're interested!
|
169521437 | 11 days ago | Gotcha, thanks for the clarification! In that case, the correct tag would be `access=private`, combined with `private=employees` to precisely specify who it is private to. As a bonus, I also added in the full set of operator tags to the object, to make it unambiguous whom the employer must be. (There's also `access=government`, but its not formally documented, much less common and less precise since this space is specifically open to employees of the Blacksburg Parks and Rec department operating the park rather than any government official.) I made the fix in changeset #169697721 , along with fixing the typo in the benches' `seats` capacity (set to `e` rather than the presumably-intended `3`): https://osmcha.org/changesets/169697721
|
169518632 | 11 days ago | Hey, I noticed while fixing the access tag in changset #169521437 that the bench seats added here were also flagged, in this case with an invalid value for seats ("e" instead of a positive integer number, which appears to just be a typo). Given these are approximately 1.8 m / 6 ft wide which yields a typical capacity of 3 persons, "3" is the most likely intended number given its position directly adjacent to "e" on the keyboard in the same finger-sweep, and is also the easiest to mistake for it given the similarity in glyphs, I went ahead and set it to this value in changeset #169697721 https://osmcha.org/changesets/169697721
|
169521437 | 14 days ago | Hey, FYI access=designated is meaningless (as it refers to explicit designation for a _specific mode of transport_) and should not be used. Per the wiki: osm.wiki/Tag:access%3Ddesignated > NOTE! The exact key/value combination access=designated should never appear on an object. The value designated must be used with a specific mode of transport. Examples: bicycle=designated or foot=designated. Given the inherent non-meaning and lack of context clues, I'm not sure what you meant here such to suggest something instead—could you clarify? Thanks!
|
169415756 | 15 days ago | Hey, thanks! Seems though that `crossing:markings` was set to `ladder` rather than `zebra` on the node (which seems a bit odd that VT would revert to using the old ladder markings rather than zebra that they use everywhere else at least for newly-painted crosswalks). Also, I see it wasn't updated on the way, leaving them out of sync. Which one did you intend? Thanks! |
169419783 | 15 days ago | Ah, in that case it goes in `loc_name`, then; see osm.wiki/Names#Local_names_(loc_name) > loc_name=* is for the name of a feature as it is known locally, but only where this is deemed to be too much of a slang name or otherwise unofficial-sounding. |
169415756 | 16 days ago | Also, one observation: I see you renamed the "Orange Building" to "VT Parking Services", and I noticed there was another larger building directly north of it also named "Parking Services". The official VT map: https://campusmap.aws.gis.cloud.vt.edu/ (which I don't use directly for mapping, just citing as a reference here) names the latter as "Parking Services Building" and the former as Residence (Orange House), which it at least appears to be built as (building=detached) per Streetside, etc. To confirm, is former building also explicitly named Parking Services per your survey?
|
169415756 | 16 days ago | One thing I needed more info from you on: the new crossing added here was tagged `crossing=uncontrolled` and `crossing:markings=yes`, but the type of markings (most likely `zebra`, as used near-universally elsewhere at VT) was left unspecified, and no markings are visible on any available source of aerial or ground imagery (usable with OSM or otherwise). Could you confirm that zebra markings are present here? Thanks!
|
169415756 | 16 days ago | Also, in the same changeset ( https://osmcha.org/changesets/169441727 ) as a further improvement I added surface and foot/bicycle access tags to the sidewalk per local convention, as well as complete crossing tagging in line with standard practice.
|
169415756 | 16 days ago | Hey—thanks for adding this! I did notice a few issues, which I fixed where possible in changeset #169441727 along with some further improvements and other nearby fixes. FYI, the specific things I noticed included:
|
169419783 | 16 days ago | Hey—I noticed the added building wasn't actually given the name "HAE Pods" (or `loc_name`, etc). Was that intentional?
|
169372828 | 17 days ago | See changeset #169374037 for the fence, track and landuse improvements to the actual western section of Heth Farm
|
169372828 | 17 days ago | *Eastern section, oops
|
169243060 | 19 days ago | Quick tip—in Rapid (which has the same datasets as MapWithAI for JOSM that I use), the "Commonweath Virginia Addresses" dataset that you're using here (source=esri/Virginia_address or esri_Virginia) has addr:county instead of the more complete, useful and locally/nationally accepted addr:city and addr:postcode. The "United States Addresses" dataset (source=esri/USA_NAD_Addresses or esri_DOT_addresses), by contrast, has essentially identical coverage (at least locally and likely statewide) but with addr:city and addr:postcode instead of addr:county, so you probably want to use that instead.
|
168684088 | 26 days ago | Also, change terrace unit to correct type instead of apartment |
168684088 | about 1 month ago | *And also add/merge complete address tagging
|
168325430 | about 1 month ago | Done in changeset #168420335 ! https://osmcha.org/changesets/168420335
|