Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add places of worship #910

Merged
merged 7 commits into from
Sep 22, 2023
Merged

Add places of worship #910

merged 7 commits into from
Sep 22, 2023

Conversation

wmisener
Copy link
Collaborator

@wmisener wmisener commented Aug 2, 2023

Fixes #861.

This PR adds places of worship. The icon is based on the classic US atlas style of a square topped with a religious icon. The initial PR contains the places of worship of the 8 religions given an icon in Carto: Buddhist, Christian, Hindu, Jewish, Muslim, Sikh, Shinto, and Taoist. The symbols I chose are:

  • Buddhist: wheel/dharmachakra poi_pow_buddhist
  • Christian: cross poi_pow_christian
  • Hindu: om poi_pow_hindu
  • Jewish: star of David poi_pow_jewish
  • Muslim: star and crescent poi_pow_muslim
  • Shinto: torii poi_pow_shinto
  • Sikh: khanda poi_pow_sikh
  • Taoist: yin-yang poi_pow_taoist

Places of worship do not necessarily fall neatly into any of the defined POI color categories. The initial PR uses attraction-brown, but there were suggestions in slack discussions for infrastructure-blue or perhaps its own color (maybe gray?). I'm open to suggestion on colors.

Other religions could be considered. Some other common religion values in the United States include unitarian_universalist (518 place_of_worship) and bahai (79). This PR also lacks distinct icons for LDS and Jehovah's Witness places of worship, which are rendered with crosses. Distinct renderings are blocked by OMT, which does not contain any denomination information that could distinguish these from other Christian POWs.

Samples:
Christian and Hindu (localhost link):
Screen Shot 2023-08-01 at 10 53 33 PM

Buddhist, Christian, Jewish, and Shinto (localhost link):
Screen Shot 2023-08-01 at 10 59 38 PM

Christian and Muslim (localhost link):
Screen Shot 2023-08-01 at 10 57 28 PM

Buddhist, Christian, and Taoist (localhost link):
Screen Shot 2023-08-01 at 11 03 09 PM

Christian and Sikh (localhost link):
Screen Shot 2023-08-01 at 11 04 16 PM

Legend sample:
Screen Shot 2023-08-01 at 11 11 31 PM
I opted for the uniform "place of worship" rather than attempting to name each religion's.

Jerusalem Temple Mount (localhost link):
Screen Shot 2023-08-01 at 11 19 37 PM

@github-actions
Copy link

github-actions bot commented Aug 2, 2023

PR Preview:

Sprite Sheets:

Sprites 1x
Sprites 2x

@ZeLonewolf ZeLonewolf mentioned this pull request Aug 2, 2023
@ZeLonewolf
Copy link
Member

Thanks for taking this on!

There was a Slack discussion that discussed PoW icons and in summary I think we need to:

  1. Use the "community anchor blue" color scheme for PoWs
  2. The other-than-Christian symbols are too fine and the features should be thickened a bit so that they have the same effective visual weight

There was also some discussion about modifying the base rectangle size if needed for effective size contrast between the symbol and the base.

@wmisener
Copy link
Collaborator Author

Updated the PR to color the symbols blue and add thicker lines for the Star of David. I'll continue iterating the symbol relative sizes via slack, though I can also post the test screenshots here if anyone desires

},
sprite: "poi_pow_christian",
color: Color.poi.infrastructure,
description: "Christian place of worship",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

id-tagging-schema calls this tagging combination simply “Christian church”, and there are other religion-specific descriptions for other religion values.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm open to this, and considered it for the initial PR, though I worried it gets into complicated terminology issues quickly. For instance, not all amenity=place_of_worship + religion=christian are churches (like LDS Temples or even holy wells), some sects of Judaism call their houses of worship 'temples' or 'shuls' instead of synagogues, what iD labels as "Sikh temples" are actually called gurdwaras, etc. But maybe these are small potatoes compared to the improvement in how understandable the legend would be.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That’s true, your point probably matters more in the context of a public-facing map like Americana than something internal to OSM like iD. The diplomatic “place of worship” terminology isn’t inaccurate, just wordy, but we have enough room for it.

@ZeLonewolf ZeLonewolf merged commit 5842f91 into osm-americana:main Sep 22, 2023
4 checks passed
@wmisener
Copy link
Collaborator Author

Ah thanks for going ahead and merging, I've been busy with work lately and haven't had much time to work on this. I think as some of the comments here and on Slack have noted, if it turns out "in the wild" that the religious symbols are too small, it could potentially be useful to make the squares slightly smaller and the icons slightly bigger. I'll also open separate issues to track other potential religions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Places of worship
4 participants