Overlapping fairways and greens is bad, but so is "almost touching". What you should be doing is *sharing* the nodes between the green and the fairway. Take a look at the wiki (leisure=golf_course#Common_mapping_pitfalls) for examples. I've gone ahead and fixed way/1361164081 if you want to look at that for an example.

Thanks for helping with my golf course cleanup project!

Thanks for helping with my golf cleanup project!

Please don't break existing fairway relations. You need to read the golf_course wiki first before you make any changes that undo the work of others. Thanks.

Hi Stykman17, thanks for your contributions. In case you weren't aware, fairways that completely enclose objects like fairways and sandtraps should be turned into multipolygons. See leisure=golf_course#Common_mapping_pitfalls

Thanks.

Hi eschack50, thanks for your contributions. In case you weren't aware, fairways that completely enclose objects like fairways and sandtraps should be turned into multipolygons. See leisure=golf_course#Common_mapping_pitfalls

Thanks.

Hi RoadHog56, thanks for your contributions. In case you weren't aware, fairways that completely enclose objects like fairways and sandtraps should be turned into multipolygons. See leisure=golf_course#Common_mapping_pitfalls

Thanks.

Hi Daskey, thanks for your contributions. In case you weren't aware, fairways that completely enclose objects like fairways and sandtraps should be turned into multipolygons. See leisure=golf_course#Common_mapping_pitfalls

Thanks.

Hi there jemiller,

There are some problems with your golf course mapping and I wanted point them out so that you can correct them and map future golf courses abiding by community standards. A very important wiki to read is leisure=golf_course.

Two of things I noticed off the bat are that your fairways are intersecting and partially overlapping the greens. There are two ways to properly map these. One is to butt the fairway to the green and share the boundary nodes between the two lines. The other is if you can see a fringe around the green that is similar to the fairway, you should completely surround the green with the fairway and then merge them together by selecting them both and right clicking and selecting merge.

The other problem is that you are naming the elements "Hole 18" and such. Those aren't the actual names and your shouldn't use names to describe features. Instead you should use the "ref" tag and set it to "18" and leave the name tag unset.

Please let me know if you have any questions.

Thanks for turning greens/fairways into proper multipolygons!

Not cool. Don't do this again.

Hey there ER, thanks for working on golf course contributions for OSM. We appreciate the map getting filled out.

There is a problem with how you're mapping though (e.g. way/1363133822). You are making the fairway (and rough) intersect/partial-overlap with the green which isn't how they are supposed to be mapped. You either need to butt the fairway up against the green and share common nodes, or you need to surround the green (if there is visible fringe matching the fairway) and turn the fairway/green pair into a multipolygon ("Merge" features in iD). Please see the wiki for more information: leisure=golf_course and feel free to reach out with any questions you have. Thanks.

Hi golfboy,

Why did you add a second, nearly identical green here: way/1363208624 ?

Also, you put two fairway areas next to each other. These should be a single object and the fairway/green should be combined/merged into a multipolygon. Check out the wiki at leisure=golf_course for how to properly map these together. Let me know if you have any questions.

Thanks for getting back to me mech32. Can you confirm with me that you understand how relationships and multipolygons work? It wasn't quite clear from your comment that you understood that and how to avoid breaking them. Thanks.

On Hole 11, you shouldn't use multiple polygons to make up a single fairway. Read the wiki (leisure=golf_course) for how to map bunkers and fairways and green properly using multipolygons with inner/outer relationships. If you have any questions, please feel free to reach out and I'll help clarify.

Hey there mech32, thanks for helping out with golf course mapping. We appreciate the work, but I wanted to point out some issues.

You have breaking relationships between features. This is caused by you deleting something like a fairway boundary that is part of a multipolygon and then recreating, but leaving the relationship around with only the green as part of it.

It's a bad idea to delete features and recreate them from scratch. They should be modified if at all possible. If this is too difficult with the iD editor, maybe use something like JOSM. And if you must delete something, make sure it isn't part of a multipolygon first and if it is, properly recreate the multipolygon after your edits.

Please read up on leisure=golf_course#Common_mapping_pitfalls to better understand. If you have any comments, please reach out and I'll clarify. Thanks.

Hi Averytt10, thanks for contributing to OSM and improving golf course maps. I wanted to let you know about some best practices before you do any more mapping.

You shouldn't create bunkers that intersect with fairways. By doing so, you are saying that any area that they overlap is somehow considered both sand and grass, which doesn't make sense. The fairway needs to be adjusted to no intersect. Depending on what you see in imagery, the fairway should go around in such a way as to include or exclude the bunker. If the fairway completely surrounds the bunker, you need to combine the two objects into a multipolygon.

The same thing can be said about greens and fairways. They shouldn't be intersecting and partially overlapping.

You might want to read over the wiki to see examples of how to map golf course features: leisure=golf_course

Thanks.

Thank you! Thank you for reusing old features instead of deleting and recreating. I've been chasing down others this morning that have been breaking things when they delete features improperly.

Good morning J,

It's a bad idea to delete golf course elements that you simply want to modify the geometry of. It's best to preserve the history and attributes whenever possible. For example, you deleted the fairway on Hole 10 and then recreated a nearly identical object. The problem is that the fairway was part of a multipolygon relationship the included the fairway and the green, but now that relationship is broken and the green remains in an error state.

Please be careful not to break relationships/multipolygons in the future. Or if you do, make sure you recreate them in the end. Thanks!

Those are some good looking golf course edits! In chance you want to help me with more golf courses? Check out mpr.lt/c/50926