Gregory Peony's Comments
| Changeset | When | Comment |
|---|---|---|
| 154175273 | over 1 year ago | I undeleted ways and intergrated them into the map.
|
| 52886114 | over 1 year ago | This changeset was made a long time ago now, so you may already know this, but please don't represent fords this way, because it can break routing e.g. if you choose to route from one side of the river to the other a router may guide you all the way around via a completely different route, since it doesn't recognise this as a valid one. |
| 154175273 | over 1 year ago | WAY: 555942026 & WAY: 1302054689 should have been modified and not deleted. The deletion of WAY: 1302054689 almost resulted in a disconnection in the highway network between the east and west sides of this village. I think that the safest bet would have been make WAY: 1302054689 an intermittent river and then fords could be mapped across it.
|
| 154171545 | over 1 year ago | The features I flagged appear to have 'dog legs' i.e. their continuity is not proper. You can use the inline validation paint style in JOSM to help you visualize continuity amongst other things. Use the URL you can find in JOSM to find the Github repo and change it to the updated version which allows you to choose what is visualized.
|
| 154175772 | over 1 year ago | I resolved the problem by undeleting the highway and conecting it with the network while aligning to imagery and making modifications. You can see this by checking its history after clicking on the element.
|
| 154175772 | over 1 year ago | I accept your apology ;) You're welcome for the comments. I have not rectified these problems, hence the "Unresolved" tag. I think that the best way to fix the problem in this situation is to use the undelete plugin, because there is only one way that was deleted, and I've already modified some of the ways which were also modified in this changeset.
|
| 154208243 | over 1 year ago | You did not provide a reason in your changeset comment for deleting these elements; I'd like to know why you did. It may be obvious to you why you did so, but others have to work it out. Deletion should mainly be used to remove elements which no longer exist, or never did. Generally speaking it is better modify elements if they represent a feature visible in imagery, especially if your intention is for those features to not be mapped again. If the geometry of a feature is accurate but you dissagree with the tagging, consider simply changing the tags, rather than deleting accurate geometry.
|
| 154175772 | over 1 year ago | You should not have deleted WAY: 555932000, especially since you simply replaced a part of it with WAY: 1302677315. In doing so you made the history and metadata of the original element significanlty more difficult to track, and they will likely be lost, unless someone with the required skills retrevies them. Instead you should have split the highway into segments, and retained the history on the segment you wished to keep, given that you replaced the highway with one of the same classification as the first version of the original.
|
| 154179514 | over 1 year ago | You're welcome.
|
| 154179277 | over 1 year ago | Sure thing. I left some comments on the tasks. Contributors can see these discussions if they open the task change-sets via the task data also.
|
| 154149947 | over 1 year ago | The ways I flagged should be connected.
|
| 154149947 | over 1 year ago | The residential areas look fairly good. Ensure that you envelope all nearby buildings as per the project instructions X<50 m.
|
| 154149821 | over 1 year ago | Virtually all of the residential highways are incorrectly tagged. Rural areas like this rarely have residential roads becuase many settlements don't even have enough roads for any to be considered residential. The villages to the SW have residential roads. Roads connecting settlements to other settlements are not residential. just because buildings are along a road that does not make it residential.
|
| 154179277 | over 1 year ago | The residential areas in this case should be joined especially becasuse they both have place=village nodes. The presence of the nodes means that they can be recognised as seperate settlements even though their sprawl has resulted in the once seperate residential areas becoming one (the residential area just north of WAY: 282589375 is not visible in this change-set). Aside pleas provide a comment with your change-set. I don't always get it right myself but here you are adding and modifying residential areas but you simply use the default project comment which only mentions highways.
|
| 154179514 | over 1 year ago | WAY: 626485122 is in fact a dry river/stream look to the NE at various imageries to see the water body it is connected to. Look to the SW tertiary highway to see its similarity with the waterways which cross it. This is quite a flat area but using a topo map helps also. I have rectified its tagging. The highway connections are not correct. WAY: 532907218 going through the village to the SW does not represent a continuous highway and passes over several barriers. I recommend referencing other imagery sources e.g. mapbox in this case as highways may be easier to identify.
|
| 154179355 | over 1 year ago | These two residential areas should be combined into one as per project instructions. The nearest buildings are 49 m away from one another. The southern residential area should also include the cluster of buildings to the West of it. I agree with the change of classification of WAY: 532907211 from service to unclassified as it connects settlements via the tertiary road to the East, though it should be more accurately digitized, becuase it currently crosses barriers and building footprints visible in aerial imagery.
|
| 137290152 | over 1 year ago | Please, please, please always provide a reason for deleting features and changing tags. Now it might take quite some effort to figure out exactly what was done here to ensure that correct data has been preserved.
|
| 135114038 | over 1 year ago | Building should be square and slightly shorter.
|
| 135116711 | over 1 year ago | Residential areas should not share common nodes with buildings, or highways.
|
| 135116762 | over 1 year ago | Residential areas should not share common nodes with buildings, or highways.
|