Since the wiki page for junction=yes does explicitly say these nodes are incorrectly tagged, I do plan to re-revert these nodes, unless someone can provide a reason not to.
---
#REVIEWED_BAD #OSMCHA
Published using OSMCha: https://osmcha.org/changesets/149827539

And they should be tagged as leisure=swimming_pool (as well as access=private for private ones), not natural=water.

Hi, private swimming pools should not be named, and name= is not for descriptive names.

This has already been reverted, discussion is occurring at https://community.openstreetmap.org/t/renewables-energy-project-source/111664

You appear to have accidentally removed a node from w331092501, and then added it back in as part of cs149854604

Hi, you've got a wetland relation (r10108830) that contains a water way (w731004638). This is conflicting information, can you confirm which is the correct one and either remove the water tags from the way or remove the way from the relation?

Hi there, you've added a fairly large line as a generator, this seems incorrect, can you please review and fix.
---

Published using OSMCha: https://osmcha.org/changesets/149856682

Hey there, where you've got the school and kindergarten next to each other, you can use the same nodes where their borders meet to make it cleaner.
---

Published using OSMCha: https://osmcha.org/changesets/149847964

The wiki says that junction=yes by itself is pointless, it doesn't indicate anything. Tagging junction=yes means just as much as the bare node itself.

If you want to indicate directions, you need more than that, which is what a manoeuvre relation can be used for. Without having information about which ways are actually joined, how can any kind of routing algorithm know what is actually meant?
---

Published using OSMCha: https://osmcha.org/changesets/149827539

Hi there, https://victoriancollections.net.au is copywrited, do we have permission to use information from there as a source?
---

Published using OSMCha: https://osmcha.org/changesets/149807470

Hey there, do you know what data SAPPA maps uses for their road names? I can't see any licensing information in their system. It might be coming from DataSA's Roads dataset (which we have a waiver to cover the CC 4.0 BY parts we can't comply with), but unless we're 100% sure then we can't use that source.
---

Published using OSMCha: https://osmcha.org/changesets/149716956

I reckon this should just be reverted.
---

Published using OSMCha: https://osmcha.org/changesets/149634217

Hey, when adding houses and other buildings you'll usually want to press Q in iD and JOSM to square them off, and as I mentioned in your other changeset, adding addr:suburb is redundant if addr:city exists, and both of those are redundant in AU since we can reverse geocode for suburbs, states and postcodes.
---

Published using OSMCha: https://osmcha.org/changesets/149460663

Hey there, if you press Q it'll square off a closed way to 90 degree corners, these are all a little wonky so doing that will neaten them up!
---

Published using OSMCha: https://osmcha.org/changesets/149460388

Hey there, for the southern carparks, is wheelchair=no the best tag to have on there? I'd probably recommend removing that tag instead of just inverting it if there's no disabled parking (from memory there is) and it isn't exactly somewhere where wheelchairs can't go, which is what wheelchair=no is intended for (i.e. somewhere only accessible by stairs).

For the houses, they're already tagged with addr:city so addr:suburb is a double up, plus tagging suburbs, postcodes and states in Australia is redundant since it can be obtained through respective relations (pretty much 100% coverage on all 3 across AU).
---

Published using OSMCha: https://osmcha.org/changesets/149461367

Considering the roads placement within a smaller, close to country town, those factors are less important in determining a roads classification. If you actually took into consideration the roads use, positioning, compared it to other roads nearby, and even just observed those roads, they’d be more than a tertiary road. They’re the main roads connecting the town to highways, main roads northbound out of the town, and also the heavy vehicle route out of the town.

Are you able to explain why you’ve dropped the main road out of a town from primary to tertiary?
---

Published using OSMCha: https://osmcha.org/changesets/149252903

Thanks for catching that, fixed in osm.org/changeset/149222216

Hey, is this actually a full residential road or just a driveway to a couple of houses?
---

Published using OSMCha: https://osmcha.org/changesets/148528867

osm.org/changeset/148413816 has added it back in.