OptikalCrow's Comments
| Changeset | When | Comment |
|---|---|---|
| 144764716 | about 2 years ago | Valid point. When I did the first pass to specify crossing:markings type (lines, ladder, zebra) I did a query on all the existing crossing=marked in the city. I wanted to leave the existing data alone as much as possible and only add the :markings tag on top- this worked for the most part, until I came across this one node which looked like the markings had been removed. I didn't want to remove the crossing= tag as that would be removing existing data. Really the main issue here is that it is a data conflict that can only be resolved by a ground survey, which is what my fixme note is for and was added to explain the situation. I just haven't had a chance to make a trip around town to survey fixme's I've added yet, I usually do those in batches. |
| 144490078 | about 2 years ago | Hey, just so you know, I intentionally left off crossing:markings from this crossing (node/9241020432) as from aerial imagery it looks like the road got repaved sometime between Bing's image capture and the Virginia state imagery capture. I put a fixme on the node and way to make sure someone knew about this. I haven't had a chance to survey it yet, but please do not re-add crossing:markings unless you can verify with a ground survey whether or not the crossing got repainted. I know iD likes to yell and makes it easy to add, but there was a reason it was missing from this crossing. I've also been trying to specify what type of markings there are in crossing:markings, so if you come across one feel free to add that extra detail from imagery instead of just adding "crossing:markings=yes" |
| 142790174 | about 2 years ago | This changeset looks pretty good too, the only thing I would advise is that "closed=" is not really a key that's in use. To indicate a shop is closed and a building is vacant, you can change the "shop=" to a "disused:shop=". This allows you to still specify what kind of shop it was, but also makes sure that any of the third parties who use OSM data know it's not currently open, because they may not know about the tag "closed=permanently." If it's more run down and fully abandoned, you can also use "abandoned:shop" and "abandoned:building", but if it's still in good shape and the building could be re-opened with only some cleanup work then disused is the best tag.
|
| 142692244 | about 2 years ago | These edits look good- though it looks like you removed the wikidata tag. Could you add it back? Its value was Q21028360.
|
| 142629502 | about 2 years ago | Hi there- I saw you requested a review on this edit. highway=pedestrian might be more appropriate here than highway=footway. highway=pedestrian |
| 141432689 | about 2 years ago | Hey there! After asking around (because I wasn't sure what to tag it as either) it looks like man_made=cooling (man_made=cooling) is the best tag for a large HVAC unit. I don't think it will render on the map, so just be aware of this. I also don't think iD has a preset for it, so if you come across more, you'll have to manually add the tag at the bottom. However, we always strive for adding accurate tags even if they aren't rendered or preset. Thanks for your work around E. C. Glass! |
| 141738260 | about 2 years ago | Hey there! I addressed your fixme request- the tagging for a pedestrian area is instead of
That's why it wasn't rendering. It should appear on the map now! |
| 141102466 | over 2 years ago | Hi dufekin, I still do not see any listed sources for your parcel importing. This makes it very hard to verify if the data you are adding has the appropriate license for OSM. Please add a source to your changesets and verify that you are allowed to use the data, or you may be blocked again. As well, I still see multiple issues with the landuses you are adding. Once again I must remind you that legal ownership of land does not always correspond to the actual landuse and that not everything belongs in OSM. Please revert your changesets and re-upload with the appropriate source attribution. |
| 140830133 | over 2 years ago | Hey there! I noticed you've been working on changing road classifications in this area- there's been some work done to standardize how we use the classification system in the US to prioritize network connectivity rather than physical quality of the roads. Generally, the guidelines are to keep major routes like trunk roads classed as trunk throughout their entire length, even if they go from a divided highway down to two lanes. These guidelines have been written out on the wiki under osm.wiki/United_States/2021_Highway_Classification_Guidance! I recommend taking a look, as the community consensus in Alabama is to follow them rather than what the wiki may say about trunk vs primary based on physical construction. Downgrading existing trunk routes breaks the connectivity of the region (and also looks fractured when zoomed out since trunk roads render at a higher zoom level than primaries). Just keep this in mind as your changes may be reverted to restore the network! Other than that, welcome to OSM and I hope you enjoy improving Alabama's map! |
| 140504692 | over 2 years ago | Nevermind, I think that standard is more for areas that are routable anywhere in the area. I think this may be correct then? |
| 140504692 | over 2 years ago | Hey Nathan! It looks like you're mapping road areas- I believe it will render on the basic osm-carto if you use the combination of rather than the tag highway:area=yes? I've seen it work around in other areas. |
| 120068505 | over 2 years ago | Protected public lands can be tagged as boundary=protected_area with the appropriate protect_class=*. leisure=park implies there are leisure activities to be found there, that the public can visit freely. Applying it to other parcels just to force it to show up on the renderer is bad practice (see osm.wiki/Tagging_for_the_renderer) Regarding the airport property, I don't know what the best practice would be here. I would lean on either not mapping it or combining it with the surrounding landuses. Again, not everything needs to be rendered on the base map so putting a tag that doesn't accurately describe what's there just so that bit of land fills in a certain color is not good practice. |
| 90470706 | over 2 years ago | Hi there, thanks for the response. My concerns are that you are importing parcels without verifying things beforehand. Are you certain the data found on the Charlottesville GIS portal is public domain? While it may be hosted openly, we still need to verify we are allowed to use and import it. Secondly, OSM should be as much of an "on the ground" map as it is of legal parcel ownership- blindly adding parcels without regard for what is actually on the ground might be a tempting shortcut, but you end up with things I and others have brought up with tagging combinations that don't make sense, weird geometry, and misleading data. If I go to a plot of land that is labeled a park, I expect a park to be there- not a patch of forest that is technically owned by the city that they colored as "the same color as other parks" but is really just a legal distinction. And certainly not a wastewater treatment plant (like on other changesets you've done!) |
| 129399472 | almost 3 years ago | I see you have a description that sounds like a guidepost on the junction node. Could you make a changeset that removes the motorway=junction tag and replaces it with a guidepost? The link I posted in my earlier comment should guide you in how it needs to be tagged. |
| 129399472 | almost 3 years ago | I'm not sure if trail junctions need to be mapped unless there is a physical marker present. If there is, then you can add a guidepost information=guidepost or a route marker information=route_marker I'm not aware of any standardized tag for named trail junctions. |
| 126158323 | over 3 years ago | At least ask the person who changed it in the first place before editing. I don't mind if you're out there in NHD import land where things haven't been touched in years but changing someone's work without consulting local mappers sucks. I'll go out there this week and do a resurvey. I do know that at least one of these has a width tagged as four meters, which does not seem like it should be as low as a creek. I also get your point about "professional geologists" but we aren't here to blindly copy the topo maps. |
| 126134644 | over 3 years ago | Also you removed a fixme I had placed on node/9621641803 without actually fixing it. These apartment complexes need unit numbers added. Please don't remove fixmes without actually fixing what was wrong... |
| 126134644 | over 3 years ago | Hi RunTrails, Please do not use the USGS map here for stream/river classification, I've previously gone through and taken care of the classification based on aerial imagery and scouting the area. Stream vs river is subjective but these creeks get wide enough to not be easily traversed over on foot and therefore fall under natural=river. I find the USGS maps *consistently* under-classify streams/rivers when compared to the OSM definitions. |
| 121366518 | over 3 years ago | Hi dufekin, What is your source for these edits? It's impossible to know just from the imagery you've listed that way/1062911927 belongs to the town. Please list your sources and verify that they are licensed correctly so that OSM has permission to use it. |
| 105309144 | over 3 years ago | Hi Natbu2384, Please take a look at osm.wiki/Tag:highway%253Dliving_street. This type of street is very rare in the United States, and is not for regular residential roads. Usage of this tag should only be for roads explicitly signposted where pedestrians have primary access over cars. It is understood that in the US pedestrians have right-of-way on residential neighborhood roads like these, but this is not enough to classify as living_street. |