Hi,
Looks to me like in modifying the river this has made the tree area cross itself ...
See https://tools.geofabrik.de/osmi/?view=areas&lon=116.06308&lat=-33.93491&zoom=16&baselayer=Geofabrik%20Standard&overlays=duplicate_node%2Csingle_node_in_way%2Cduplicate_segment%2Cway_in_multiple_rings%2Cintersection%2Cintersecting_segments%2Cring_not_closed%2Ctouching_rings%2Crole_should_be_inner%2Crole_should_be_outer%2Cinner_with_same_tags%2Cways%2Cduplicate_node%2Csingle_node_in_way%2Cduplicate_segment%2Cway_in_multiple_rings%2Cintersection%2Cintersecting_segments%2Cring_not_closed%2Ctouching_rings%2Crole_should_be_inner%2Crole_should_be_outer%2Cinner_with_same_tags%2Cways

You should fix this ...

Ok. Now it is one simple way. ( In JOSM that I use the command would be 'Combine' and that would need the two ways to be touching at one or two nodes, not overlapping one another),

As a 1 member relation .. the tags can be placed on the way and the relation deleted.. I have done this as it is far simpler.
See way/306701792

Hi
The sign a the top saying 'Lorna Pass' points to the pass .. it is not the name of this rack...

Hi,

Welcome OSM!

Multipolygon relations are difficult. In OSM the outer ways are not allowed to share segments. For the apron here I's simply make the two ways into one simple closed way thus no need for a relation.
Left me know if that does not make sense.

Hi,
Welcome to OSM ... one issue from this changeset ...

Multipolygons outer ways should not share segments. So the parking relation is wrong.
Why are the parking areas 2 separate ways anyway? The easy thing to do is to make them one simple closed way - no relation required.

Not interested. Deleted.

Hi,
Welcome to OSM!

The tag 'area=yes' is not usually required.

We all do those mistakes .. just different kinds to stop the boredom. :)

Hi,

Yer .. that Park relation will not 'work'.
First each member of the relation needs a 'role', 'inner' or 'outer'.

Second ... the Way: 83407591 "landuse"="farmyard" is a problem as it presently causes the boundary of the Park to cross itself - that is not on. I'd 'cheat' and make the Park boundary not cross itself by extending it a few meters to fully enclose the farmland or retract the Park boundary a few meters so it does not full enclose the farmland.. your choice.

Hi,
The warning of 'unclosed multipolygon' for relation 14998733 South Fremantle Terminal should be fixed. ... The tag 'barrier=fence' can be placed on the relevant individual ways. The gate across the entrance road could be extended to the fences on either side...

Hi,

Looks like an error?
way/1121223120, highway=service bridge=yes, layer=1 ... that is quite a long bridge crossing a main road..
Possibly a gas pipe line road? In which case I'd map it as a track.???

great ... thanks

Hi,
Tagging these areas as 'leaf_type=leafless' mean there are no trees with leaves there? I would think these areas would be of mixed types?

Hi,
way/1116245787 and 1116245786 have no tags. construction?

Hi
This left behind the tag construction=cycleway. That should be removed if construction is finished?

Hi,
Would this be due the weather we have had recently? If so repairs could be expected eventually?

The Maxar imagery looks to be the most upto date. The DCS Base Map shows the legal boundaries.

Tracks made by construction work should not be mapped - they will go away with occupancy and landscaping, so they are not 'permanent'.

Unfortunately the past edits have not been good. Better to start smaller like this edit and see how you go. Maybe map the houses?

Hi
The tag building=* should only be for the building, not a collection of buildings.

Hi,
I have deleted the Chinese 'names' from this and other past changesets.
Please continue mapping .. but consider the 'names' and possibly use the description tag.

Hi
The tag building=* should only be on a building, not on an area that contains buildings.