pkoby's Comments
| Changeset | When | Comment |
|---|---|---|
| 61866165 | Hi AParsley, Great job on the additions. However, I noticed that it looks like you're using Bing Imagery as your background layer. That's pretty badly offset, and also not very sharp. I'd recommend using Mapbox Satellite. It aligns with GPS traces pretty well, much better than Bing. |
|
| 58590398 | I understand that it is not a public path. The fact remains that the path exists, and OSM is about mapping what is there, not what should be. The line should remain on the map as long as there is a path on the ground, and there are tags to add to it to make it clear that it is not public access. If access=private isn't enough, put on access=no. It would be the fault of a routing engine if it allows access through a road tagged that way, and I have yet to come across one that does route on a private road. I would think that a person riding or hiking around here might see the path on the ground and see nothing on the map, assume it's okay to hike. So keeping it in the database with a clear tag stating it is not public would, I think, prevent careless trespassing. |
|
| 58590566 | If the path exists, it should be on the map, even if it's unsanctioned. It is not clear from external sources if it is private or not, so I'm not adding any access tags to it at this time, but I have surveyed this in the past, and the path exists. |
|
| 58590398 | Whether or not it's private doesn't change whether it exists. I'm reverting the changeset. Based on https://gissites2.centrecountypa.gov/java/centregis_public/ and http://centrecountypa.gov/index.aspx?NID=424, the land is Vacant, which is hard to declare as private, but for now I'm putting an access=private tag on it. |
|
| 57955180 | I understand. I was not aware of the distinction between the campus and ag-sci area. I've removed the latter from the former, and added the ag-sci area to the super-relation of Penn State that includes all the campuses. Sorry for the misunderstanding. |
|
| 57955180 | That's a fair point. I didn't realize that the geocoders had trouble. Although, might that be a problem with the geocoders and not the database?
|
|
| 56355949 | I haven't been to Bellefonte in awhile; is the building next to the YMCA actually gone, or has the pub just moved to a different one down the street? |
|
| 52642014 | Hi! It looks like you probably meant to add car parking here. This big a bike lot would be awesome, but I don't think it's right.... |
|
| 52024815 | Whoops. The dataset I was working from to compare had those keys, and I forgot to remove them. Thanks for pointing it out. I'll edit. |
|
| 51895031 | Seems it was fixed. I'm not sure what happened there. But it's all good now. Thanks for pointing it out. |
|
| 50330693 | Most of the construction is done, but until they remove the orange barrels, I'm going to assume (and hope) that they might still improve the painted lanes. So until then, it's probably premature to add lane information. |
|
| 47186966 | Hi Jim, Welcome to OSM. I want to point out that OSM is a database of current information, so the points you added for old train stations aren't really something that you should add. There might be better tags you could use if you want to put that information in, though, such as demolished=*: (so you could add "demolished:building=train_station"). They're not really attractions, after all. Thanks! |
|
| 47073478 | So I saw the node to fix sidewalks. I don't really know how, though. The service road is first and foremost a road, even though the N/S sidewalk connects to the NW/SE sidewalk via the piece of road. I checked high-res PSU imagery (I should show you how to set that up if you're not using it yet...) and Mapillary photos. For now, I put a "foot=yes" on the bit of service road that connects the sidewalks. Also, thanks for checking the bike racks. I have a goal to add/verify all racks on campus in the near future. I saw you added capacity for some, which is great. I would also add "bicycle_parking=*", with things like "=rack" (the coat hanger racks and wavy snakes) which is what almost all are on campus. For capacity, I count the coat hangers as each gap plus two ends (so 10 for the ones with 7 hangers), and the wavy racks the same way (both up and down gaps plus ends). |
|
| 46684362 | Hey MG, Just wanted to point out that the name=* field is used for the actual visible name of things. For the hydrant, for instance, if it said "Disconnected" on it, that would be accurate, but since I assume that you're trying to indicate that the hydrant is inoperable, I would suggest adding a "Lifecycle prefix", so perhaps disused:emergency=fire_hydrant would be an option. As for the ruins, if they're not actually named, perhaps you could put that information in a description=* tag. Happy mapping! |
|
| 46384962 | Your reasoning makes a lot of sense. In my way of thinking, zebra stripes are uncontrolled in the sense that they aren't unmarked and don't have signals. But taginfo shows crossing=zebra to be one of the most common, so I'm going to say it seems like a good strategy. Yes, I suppose as long as the way is highway=footway, footway=crossing, and the node is highway=crossing, the crossing=* isn't important. |
|
| 46384962 | Hey, I noticed that you changed a lot of crossing=* to crossing=zebra, but according to the wiki, this isn't a standard tagging scheme. It appears it's a commonly used tag, but while I think it might be fine changing =uncontrolled to it, I think changing =traffic_signals to =zebra removes important information. The wiki suggests using crossing_ref=zebra in these instances, while still using =uncontrolled or =traffic_signals or whatever. Would it be possible to revise these changes? |
|
| 46230251 | Awesome! I was just talking about this yesterday, where there's all these names for subdivisions, and no one outside of the residents ever knows where they are. |
|
| 43195140 | Reverted both changesets. |
|
| 43195140 | You really shouldn't delete ways with lots of history. Any reason you can't realign the existing ways to a GPS trace? I think this changeset (and 43195172) should be reverted. |
|
| 41672040 | Just want to check that this path is actually not there? Your comment says you added a revision, but it looks like you just deleted it. So the Bailout trail ends abruptly. I want to make sure this wasn't an accident. Thanks! |