aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 79604694 | almost 6 years ago | Cool I see you added the species tag now, that's good. Even with the wikidata tag I think it's helpful to add the species tag, but with both wikidata and species it's even better. The point I was making though is it should be `species:wikidata` not `wikidata`. For example this tree has its own wikipedia page and wikidata value https://en.wikipedia.org/wiki/%C3%81rbol_del_Tule and https://www.wikidata.org/wiki/Q252182, but in your case there is no wikidata for this exact tree, only its species so it should be species:wikidata=Q... |
| 79604694 | almost 6 years ago | Please could you let me know when you've fixed this or if you need a hand? |
| 79604694 | almost 6 years ago | Hi, thanks for mapping tree species. Please see species=* + the genus and taxon tags. Also for tagging with wikidata you'll need to use species:wikidata unless the tree is notable and has it's own wikidata item. |
| 78808859 | almost 6 years ago | I've sent a waiver request to ECAN to ask for imagery tracing permission, but let's not expect that. balcoath it looks like you've reverted your affected changesets, did you manage to do them all without issues? Let me know if you still need a hand.
|
| 79373093 | almost 6 years ago | Hi Seb, I noticed you just added the name Stand A to this node node/1263263479 which has ref C. Looks like they start at A at the top and work their way down so should that be Stand C? |
| 79213300 | almost 6 years ago | looks like changeset/79253913 may have reverted some of your changes but I don't have enough local knowledge to know. You can see a diff at https://osmcha.mapbox.com/changesets/79253913/ |
| 76259723 | almost 6 years ago | I think it's somewhere in between motorway and trunk, it's mostly only access by ramps (with some exceptions for very minor service roads only) but no roundabouts or traffic lights, it has the higher 100km/hr speed limit indicative of a motorway, but it's not officially designated as a motorway (you'd see start motorway signs if it were). So I'm okay with it being trunk even though I think motorway is also fine. The main things I was trying to help you with were 1. where you removed oneway=yes from entry/exit ramps. 2. where you removed the _link classification from entry/exit ramps. > Anyway, just reset whatever you want, I won't touch things anymore. If you see things which are wrong or missing, please do fix them. |
| 79253913 | almost 6 years ago | hi with node/1832386691 it looks like you've just added it back as a member of the Bankstown Line relation. However changeset/79213300 just removed it one day ago since the line no longer uses that platform. Are you sure you're correct, if your unsure could you revert this? |
| 76259723 | almost 6 years ago | See https://osmlab.github.io/osm-deep-history/#/way/355252140 where you removed the oneway=yes, but I've fixed this now. By the way "Princes" is correct not "Princess". Sorry but if you set avoid motorways then you can't really complain when it avoids this motorway! Not all motorways are toll roads, if you want to avoid toll roads set avoid tolls not avoid motorways. This section has no roundabouts, so I and although it's not named motorway I think it fits OSM's motorway description. |
| 76259723 | almost 6 years ago | I fixed the _link classifcations. |
| 76259723 | almost 6 years ago | I fixed the oneways, could you fix the _link classifications? |
| 76259723 | almost 6 years ago | I noticed you've also remove a few oneway=yes from exits/entrances and remove _link classification which is incorrect. I'm fixing this for you. Could you explain what you were trying to change in your changeset? |
| 76259723 | almost 6 years ago | This is looking more and more like a motorway now since the removal of at grade intersections, the increased speed limit, lack of traffic lights now. |
| 79070571 | almost 6 years ago | This is not the correct way to fix this... probably better to just omit the name completely. |
| 79070634 | almost 6 years ago | why not use alt_name? name_1 is deprecated name=* |
| 79092954 | almost 6 years ago | I reverted your change because the wikidata entry you used is for a bay ie a body of water. The OSM feature relation/1214649 already has that wikidata id, this relation is for the administrative boundary not the body of water. |
| 79179920 | almost 6 years ago | just a tip, although not essential, it's best practice to try and use the existing node as part of the new way and transfer the tags across, that helps retain the history a bit better than deleting the node completely. it's not always possible though, or easy so again not essential. |
| 79192488 | almost 6 years ago | Top job! |
| 78971931 | almost 6 years ago | no worries! |
| 78971931 | almost 6 years ago | I've fixed the format, it should just be en:Title not the full URL. |