ACarlotti's Comments
| Changeset | When | Comment |
|---|---|---|
| 88641827 | over 5 years ago | Is there pedestrian and cycle access past this closure point? If so, then you need to add a short length of cycleway to connect this up. |
| 88616839 | over 5 years ago | Hello Bazzagazza101, and welcome to OpenStreetMap!
|
| 88529193 | over 5 years ago | It would be better to split up changes like this into separate changesets (one for each golf course affected). As it is, the changeset description here implies that you're only changing one golf course, which together with the large changeset bounding box suggests that you could have made some accidental changes elsewhere (though having looked more closely, I don't actually think that was the case here). |
| 88525807 | over 5 years ago | Apparently this bridge is not officially open yet (but in practice it is now being used). |
| 88314498 | over 5 years ago | I don't know how you reshaped it, but your changeset definitely created a multipolygon where there wasn't a multipolygon before. I've restored this to the original polygon, and deleted the node that was shared with the powerline.
|
| 88314498 | over 5 years ago | A few comments.
|
| 88421271 | over 5 years ago | I suggest mentioning buses in your changeset comments (e.g. "Update bus route 45a"), so that other people know what you're editing without looking at the contents of the changeset. (I thought it might be building numbers at first). |
| 87096896 | over 5 years ago | This changeset also contains a misinterpreation of aerial imagery. The outline you've drawn for the middle building is clearly showing the outline on Bing imagery, which was not taken directly overhead. The actual footprint on the ground is smaller - you've effectively mapped the roof in a different location to the ground floor, and shown it all as a single outline. Also, I'm pretty sure there is some stuff between this middle building and the large one to the right, though I can't tell what shape it is. (So it could be a building filling the entire gap, or it could just be one or two covered walkways). If you can't tell what it is, then it might be worth adding a note to the map so someone else can fix it later. I think this is all the feedback I have to offer for now, but feel free to ask me more questions. |
| 86784118 | over 5 years ago | I think there are some issues here too. The building to the west of High Street in particular looks as though it has been drawn by tracing around the approximate shape of a roof as viewed from an angle, and different roof heights and shadows might have been misinterpreted.
Also, if you're editing an existing building, it is almost always better retain the original OSM feature, and just edit the tags and move or replace the nodes as needed. This makes it much better to see the history of the building, and identify it as something that was already mapped, rather than something you're mapping for the first time. |
| 88362897 | over 5 years ago | I don't know how to do it in iD, but the wiki lists some ways of doing it in other editors:
Also, there's lot of other useful advice on the wiki. Some pages that might be particularly relevant to you are:
|
| 87837818 | over 5 years ago | To conclude, this changeset seems to worsen the mapping of these buildings, and I think it should be reverted, unless you have any good reason that it shouldn't be reverted. |
| 87837818 | over 5 years ago | Hi daniel,
Additionally, the general effect of this changeset has been to move all the buildings a small (though inconsistent) distance to the south. This sort of change often means that either you or a previous mapper have used aerial imagery that is not correctly aligned. The original mapping (before your changeset) seems to be correct relative to most of the surrounding buildings, and the public GPS traces don't clearly show that the original mapping was misaligned, so in that situation I think it is best to keep the realtive alignment of features correct, rather than trying to move a few buildings to match individual buildings to match aerial imagery that could well be misaligned. |
| 88362897 | over 5 years ago | Hello Daniel, and (a slightly belated) welcome to OpenStreetMap I'm a little confused by this changeset. Firstly, this is because your changeset comment doesn't say anything useful, and certainly doesn't help me or anyone else to understand what you're doing. You also haven't given any indication of what sources you've used to take make the changes - there's a big difference between how much other contributors can trust a changeset if it's based upon someone visiting the location, compared to if it's based upon out-of-date or hard-to-read arial imagery. Another thing that would also help is splitting this up into two separate changesets, since the changes in London are unrelated to the changes in Hull. Secondly, the changes themselves seem a bit odd. The change in London appears to be a realignment of the outline of a building under the station, but this new alignment doesn't seem to follow any building edge that I can deduce from other mapping and arial imagery. You also set the layer to 2 for some reason, even though the building seems to be under the tracks (which I think are level 1). The changes in Hull also seem surprising. Has New North Bridge House been demolished? I can't see any evidence of this online. You also created new nodes for a building shared with an underground cable, which might have been accidental and is probably wrong - I wouldn't expect an underground cable to be running directly under an external wall. Can you clarify what is going on with this changeset, and fix any errors you think you've made?
|
| 88243969 | over 5 years ago | This look wrong - are you sure your sources are up-to-date? The recent A14C2H works were supposed to close up all accesses like the one you just readded, with access provided from the A1307 (or other parallel routes) instead. In the unlikely event that there is still an entrance here, it certainly wouldn't provide access from the M11, because that would require a sudden left-turn across two lanes of 70mph traffic from the A14. |
| 87969289 | over 5 years ago | You seem to have mistakenly added a node to the residential area boundary. This probably shouldn't be sharing any nodes with the track you added. |
| 87943981 | over 5 years ago | This just duplicates the tagging that's already on the building outline, and is therefore redundant (c.f. "one feature, one OSM element"). I think you should revert this changeset. |
| 87889247 | over 5 years ago | Hello Chrisgf98, Could you please add meaningful changeset comments to any future edits you make? These give a clear indication to other people (including you in the future) of what you intended to do, which makes it much easier to avoid misunderstandings, and correct any mistakes more quickly. There's more information about this on the wiki at osm.wiki/Good_changeset_comments
I also notice that three of your changesets seem to include edits made in two or more locations that geographically a long way apart. This is generally a bad idea, because it is very hard to work out what areas are affected by changesets when they have a large bounding box. (Some tools only show a bounding box, and some other tools that try to show all the changes break if the bounding box is too big). In future, could you please try to ensure that changes in different locations are uploaded in separate changesets. Thanks,
|
| 56904891 | over 5 years ago | I think the tagging is now better, but there still seem to be some inconsistencies. Firstly, you used the tag "oneway:bicycle=-1"; I think this is an error, since I can't find that tag listed on the wiki anywhere.
|
| 56904891 | over 5 years ago | Could you please review and fix the access restrictions tagged on Princess Road (and possibly some of the surrounding roads). You're changes in this changeset and changeset/56900652 (there may be other relevant changesets I haven't noticed) appear to have incorrectly prohibited access both for cars continuing along Princess Parkway, and for cyclists along the cycle route to the east (in particular along the section shared with buses).
|
| 87638632 | over 5 years ago | Should the individual ways now be tagged as "railway=construction" and "construction=rail"? This would then cause them to appear on the main rendering on the website (similar to how the line from Bletchley towards Bicester is currently shown). |