OpenStreetMap logo OpenStreetMap

Changeset When Comment
169261967

RE: way/677780399

When drawing golf course areas (such as greens, fairways, bunkers, tees, etc.), please be aware that the ways used to outline those areas can't cross over each other. Fairway outlines shouldn't cross over greens or bunkers or other fairways for example. If you could go back and clean up where you've made this mistake, that would be helpful. But more importantly, if you could stop from doing this in the future, it would be greatly appreciated. If you have any questions about golf course mapping, feel free to reach out.

169248708

RE: way/835073894

FYI, you added the fairway tag to the way mentioned above, but that was unnecessary (and wrong) since the combination of the fairway and green were already correctly tagged in the multipolygon relation. Please be careful when tagging ways that belong to a relation. It's often the case that the relation defines the object, instead of the components. Thanks.

169118861

I wouldn't say you're "responsible" necessarily, but it sure would be appreciated if you did introduce quality errors like overlaps into the map.

169175639

RE: way/1416012868

Hey there Max,

There were a few problems with this change that I will fix but wanted you to be aware of in hopes you avoid them in the future.

* Around the hole listed above, the tee box to the West of that rough had a golf=hole way that had two nodes in the tee box. The typical way to map a hole line is to just map from the farthest tee.
* The same tee box should have been excluded from the drawing of the rough section, since the tee itself is not "rough", but "tee".
* You forgot to share this node: node/13012884504 in the drawing of the rough, so the rough was overlapping the fairway (which is a Q/A check that triggered me into looking at the problem).
* The rough at node/13012884534 extends over the golfcart path when it shouldn't.

As I mentioned, I've corrected all these, but just wanted you to be aware. Thanks.

169118861

RE: way/1271590618

When drawing golf course areas (such as greens, fairways, bunkers, tees, etc.), please be aware that the lines used to outline those areas are not allowed to cross over each other. Fairway outlines shouldn't cross over greens or bunkers for example. If you could go back and clean up where you've made this mistake, that would be great. But more importantly, if you could stop from doing this in the future, it would be greatly appreciated. If you have any questions about golf course mapping, feel free to reach out.

169083805

RE: way/866775459

When drawing golf course areas (such as greens, fairways, bunkers, tees, etc.), please be aware that the lines used to outline those areas can't cross over each other. Fairways outlines shouldn't cross over greens or bunkers or other fairways for example. Take a look at osm.wiki/File:Golf.png for an example of the "Wrong" way to map a fairway and a green. (The same can be said about fairways and bunkers, tees, roughs, etc.) There are some cases where a fringe exists around a green and you should draw the fairway outline completely around a green. Other times, the fairway and green butt up against each other. In that case the fairway and green should share the same nodes at the boundary between the two. If you could go back and clean up where you've made this mistake, that would be great. But more importantly, if you could stop from doing this in the future, it would be greatly appreciated. If you have any questions about golf course mapping, feel free to reach out.

169077720

RE: way/1415475862

When drawing golf course areas (such as greens, fairways, bunkers, tees, etc.), please be aware that the lines used to outline those areas can't cross over each other. Fairways outlines shouldn't cross over greens or bunerks or other fairways for example. Take a look at osm.wiki/File:Golf.png for an example of the "Wrong" way to map a fairway and a green. (The same can be said about fairways and bunkers, tees, roughs, etc.) There are some cases where a fringe exists around a green and you should draw the fairway outline completely around a green. Other times, the fairway and green butt up against each other. In that case the fairway and green should share the same nodes at the boundary between the two. If you could go back and clean up where you've made this mistake, that would be great. But more importantly, if you could stop from doing this in the future, it would be greatly appreciated. If you have any questions about golf course mapping, feel free to reach out.

169159391

sorry

168982538

Hey there dirty,

On the 7th hole, you broke the relation between the fairway and the inner "bunker" (looks more like a rough with the trees there, but that's irrelevant). I'm trying to make sure relations like this don't get broken so I wanted to reach out and work with people that make these edits. Were you aware that you were messing with the relation? What did you think was wrong with the way things were mapped previously? Thanks.

169038944

RE: way/1415302516

Please don't share the nodes of the green if you have the fairway surrounding the green. If you can't see any fringe around the green, you should make the fairway butt up to the green and share the nodes on the boundary *between* the green and fairway instead. Please read the wiki for visual examples and instructions on how to better map golf courses: leisure=golf_course#Common_mapping_pitfalls. If you have any questions, please let me know and I'll gladly help clarify things. Thanks!

168943744

Digress away! I've focused my attention this year on cleaning up all fairway/green overlaps. I've probably tackled 12,000+ instances so far this year and am exhausted. I have about 8,000 to go. I've also set up daily diffs so I can catch people making these "mistakes" early on so that they don't spread. You can see some of my progress on my maproulette challenge: https://maproulette.org/browse/challenges/50926.

You're right that the partial overlaps are evil and need to be wiped off the map. I don't quite understand all of the math/geometry theories behind it, but somehow it is "cleaner" to have it drawn as two fairway parts instead of a single one. Ideally, the grounds crew would leave a fringe around the green and you can just draw the fairway all the way around the green and leave a gap. At that point, a multipolygon is the cleanest representation.

Thanks again for the assist.

168982148

RE: way/1414980362

Hi Mezzy,

Can you please help me understand what you're trying to do with this golf course? It appears that you are simply tracing existing objects (fairways) and creating duplicate objects. I'm going to revert your changes, but want to check in with you so I can understand and help you achieve what you set out to do, but in a way that is compatible with OSM practices. Thanks!

168943744

Hey there CurlingMan13,

Thanks for helping clean up golf courses. FYI, you can't share fairway nodes with the green nodes when you have a fairway before and after a hole like the fourth hole in this changeset. Something about it not being a mathematically simple geometry. Some editors (JOSM in particular) will warn you against that type of edit, but I guess iD doesn't. I've gone ahead and fixed it up here (way/1415176817) for you but wanted you to be aware for future edits where the green doesn't have a clear fringe around it.

Thanks.

154284461

Please don't draw cart paths as areas. Thanks.

168897586

RE: way/593951832

When drawing golf course areas (such as greens, fairways, bunkers, tees, etc.), please be aware that the lines used to outline those areas can't cross over each other. Fairways outlines shouldn't cross over greens or fairways for example. Take a look at osm.wiki/File:Golf.png for an example of the "Wrong" way to map a fairway and a green. (The same can be said about fairways and bunkers, tees, roughs, etc.) There are some cases where a fringe exists around a green and you should draw the fairway outline completely around a green. Other times, the fairway and green butt up against each other. In that case the fairway and green should share the same nodes at the boundary between the two. If you could go back and clean up where you've made this mistake, that would be great. But more importantly, if you could stop from doing this in the future, it would be greatly appreciated. If you have any questions about golf course mapping, feel free to reach out. Thanks.

168849710

Hey Pascal! Wasn't sure this was you or not. Love your tools!

I agree with everything you say. I'm a "mega active" mapper, mostly due to golf course cleanup over the last few years. I was already cleaning up the edits from the user and noticed your comments. I didn't see any "call to action", so I wasn't sure if it was intended for the mapper of this particular changeset or a trigger for others to look into it. Sounds like it is that latter.

Is there some coordinated effort that chases down these tags? I'd love to know about that if so.

Thanks.

168849710

@NeisReview: what is the purpose of your changeset comment? It doesn't provide any value afaict.

168798709

There's a few problems with way/638818957.

* You moved the fairway and green to an entirely different spot on the map
* The fairway and green are still intersecting each other since they aren't sharing all nodes on the boundary. Looks trivial, but this still sets of the automated Q/A checks that look for this error.
* The golf=hole is now all out of whack
* The golf=hole shares a node with the green. It shouldn't.

168819727

hmm, not sure how that happened. sorry about that.

168711963

Thanks for getting back to me 2019.

I agree there is no rough between the fairway and green on the 16th hole and the two should butt up against each other. You said you updated them to not overlap, but the one hole I referenced at the top of my comment is still wrong? Did you miss this one? You have the fairway overlapping the green instead of sharing the boundary nodes: https://ibb.co/SwJ4fpMz

What it needs to look like is this: https://ibb.co/GQSBd3Kx

I hope that makes sense. If not, please let me know. Thanks.