aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 89040906 | Thanks for fixing that one. A few other things I noticed: 1. There is also https://osmlab.github.io/osm-deep-history/#/way/498123668 which was already mapped at way/498123670 2. I don't think service=emergency_access is entirely correct here, these are mostly service=driveway. service=emergency_access would be for say an office building which has a separate access road specifically for firefighters. 3. https://osmlab.github.io/osm-deep-history/#/way/761474319 and https://osmlab.github.io/osm-deep-history/#/way/766373702 added the building tag, but this way is for the grounds, the building is already mapped there. |
|
| 89256197 | Thanks for the reply, that would be tagging for the renderer osm.wiki/Tagging_for_the_renderer a practice which is not in the spirit of OSM, so I'll revert the change so that these areas accurately represent what's on the ground. |
|
| 89040906 | hi, while most of the changes here look good, I noticed a couple of these fire stations were already mapped as an area covering the grounds, not just the main building, in these cases the primary tags like contact etc should go on the existing amenity=fire_station. For example you added amenity=fire_station to way/397201960/history but the fire station was already mapped at way/792923178. Did you want to fix this up, or should I? |
|
| 89113699 | this is already mapped at node/7704878760 so I deleted the one you just added |
|
| 89253393 | Would the lower frequency ones you added in the residential areas be better mapped as power=minor_line? |
|
| 89256197 | hi in all the imagery here this is wood, not sand, and seems quite unlikely these large areas would be sand. What was your source for this change? |
|
| 83088215 | hi I see you've mapped way/22588137/history#map=19/-37.84019/144.91251 as a oneway road however on all the aerial, satellite and Mapillary imagery I could see this is a two way road, what was your source for mapping it as oneway? |
|
| 89004677 | this changeset has been reverted by changeset/89005958 |
|
| 89005958 | see reasoning at changeset/89004660 |
|
| 89004660 | hi I reverted this change because I noticed it combined the city and suburb nodes, which broke both the suburb and city relations for Sydney. The Sydney place=suburb relation/5729534, similar to other suburbs should have a label node per osm.wiki/Relation:boundary#Relation_members and it should be tagged with the same place= tag as the relation. So the place=suburb relation for Sydney can have a label node also with place=suburb. The Sydney place=city (not the city of sydney lga) relation relation/5750005 also can have it's own place=city label node, and it in this case also has an admin_center node. The way you combined the place=suburb and place=city nodes broke this. Happy to discuss this further either here, or on talk-au or on the maptimeoceania slack. |
|
| 89004660 | hi since this is affecting a major feature I'll revert this first, then follow up with discussion about the change is a minute. |
|
| 88856654 | I tried to look at Mapillary imagery, but we don't have any here yet. |
|
| 88856654 | From the imagery most of them do appear to be terraces (row houses with shared walls), but it doesn't looks the whole block is one, rather there are a couple of terrace buildings each with a few homes. So the previous mapping is just a rough approximation until it is improved, saying most of these are victorian area terrace houses with 2 levels, that doesn't mean we should just delete it though. |
|
| 88856654 | hi how come the buildings were deleted here? There was important information like that it's terrace buildings, the building architecture and number of levels. |
|
| 88856862 | hi did you mean to delete the track name here? |
|
| 70470750 | I've reverted these changes due to reasoning discussed here. |
|
| 88674461 | Hi a service building per building=service " represents building specially designed to contain constantly working machinery like pumps, transformers or measuring equipment. Usually such building is not supposed for people being inside while machinery works normally." So building=commercial is a better fit for a vet where people work inside so I've changed this back. |
|
| 88645653 | Yeah it's the one on the concrete ramp I was talking about, but with it gone then you're right it's open! |
|
| 88645653 | must have only happened in the last few days then on the 24th of July there was still a barrier https://www.mapillary.com/map/im/NX3nPJVRLcH_eWl_F0qaqQ |
|
| 88615434 | *phones |