IsraelHikingMap / Site

Israel Hiking Map has maps, route planning, and travel information for Israel. This repository holds the files needed for running the Israel Hiking Map site and apps.
https://israelhiking.osm.org.il/
Other
84 stars 33 forks source link

Peak is not a POI #1875

Closed amiad closed 1 year ago

amiad commented 1 year ago

What happened?

לפי מיטב זיכרוני פסגות היו בעבר נקודות עניין עם תמונות. עכשיו הן לא מוצגות כך. רק אם מחפשים אותן אז הן מוצגות.

What steps will reproduce the bug?

כיפת עשוש לא נקודת עניין https://israelhiking.osm.org.il/map/16.39/30.4303/35.0014 רק אם מחפשים אותה https://israelhiking.osm.org.il/poi/OSM/node_5882330268?language=he

What I expect to happen

שהיא תוצג כנקודת עניין

Platform

OS Name and Version

All

Browser Name and Version

No response

Additional information

No response

zstadler commented 1 year ago

Potentially, add a peak to the nature POI layer only if it has a description or an image

HarelM commented 1 year ago

I don't think they ever were, only maybe if merged with a wikipedia/iNature value or if the peak is also a viewpoint. Usually, peaks are not interesting to visit so they were never part of the layer, much like streams I guess...

amiad commented 1 year ago

אז אני חושב שהן צריכות להיות נקודות עניין אם יש מידע נוסף. נחלים הם ארוכים ולכן לא מתאימים להיות נקודה.

HarelM commented 1 year ago

מצטער, אני לא מסכים, לא קיבלתי פה טיעון מספיק חזק כדי להוסיף והסברתי למה אני חושב שהן לא צריכות להיות (הן מעולם לא היו דרך אגב). בדומה לנקודות אחרות, ניתן לחפש פסגות, אבל אני לא רואה סיבה להוסיף אותם לשכבות.

HarelM commented 1 year ago

See here as well: #798 this is an old issue which confirms that this was a search results POI and not a regular POI.

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days

HarelM commented 1 year ago

הוספתי קוד שמציג פסגות עם תיאור או תמונה, ורוב הפסגות שמצאתי לא מאוד מעניינות. אין לי בעיה להוסיף את זה לנקודות העניין, זה לא שינוי קוד גדול, אבל אני באמת חושב שאין פה הרבה עניין למשמתשים. אולי לגבעה הספציפית הזו כדאי להוסיף עוד תגית כמו תצפית או אטרקציה תיירותית. הנה כמה שמצאתי לדוגמא שנראה שאין בהם הרבה עניין לציבור: https://israelhiking.osm.org.il/poi/OSM/node_278478102?language=he https://israelhiking.osm.org.il/poi/OSM/node_8541834517?language=he

HarelM commented 1 year ago

Relevant code changes can be found in the linked commit, currently in a branch, until goods examples are presented. Adding all possible peaks (that has a description or an image) in order to solve a single peak that is interesting is not a good UX, so I'm looking into more than just a single location where a peak is/might be interesting. To clarify - this is not a coding problem, this is a UX discussion now. :-)

amiad commented 1 year ago

מצאתי 50 פסגות עם תמונה או תיאור ובלי תגית tourism. אכן צריך להחליט אם הן מצדיקות את השינוי.

https://overpass-turbo.eu/s/1xZs

HarelM commented 1 year ago

מבדיקה אקראית של כ-5-10 נקודות מהרשימה הנ״ל מסתמן, לדעתי, שיותר נכון להעביר את המידע שבפסגה לנקודת העניין הקרובה שלרוב עליה מדברת הפסגה או להוסיף לפסגה שהיא גם נקודת תצפית, יש גם חורבות וכו׳, אבל על פניו נראה יותר כמו מיפוי לא נכון של הפסגות. אני לא נחרץ בדעתי, ואשמח לשמוע דעות אחרות בהינתן המידע הנ״ל. אשמח לקבל מספיק דוגמאות שבהן המיפוי מרגיש נכון והוספת נקודות העניין האלה נראה כפתרון היחידי.

zstadler commented 1 year ago

I found 44 peaks that have an image or a description and are not tagged with "tourism" or "historic" - https://overpass-turbo.eu/s/1y0F

It included

I think they are valuable.

Certainly no less than some of the Wikipedia POIs I happened to see near הר צפת😄 image

HarelM commented 1 year ago

Looking at the map and images: הר עצמון - נראה כמו תצפית לפי זה שעולה למקום שביל עפר הר צפת - נראה כמו מנומנט הר קנאים - אותן תמונות נמצאות גם בתצפית וגם בפסגה, כך שבפועל זה מדבר על אותה נקודה, אם לא היו תמונות בפסגה אלא רק בתצפית לא חושב שלמשתמש זה היה "מפריע" - מה שזה ייצור זה כפילות. הר בוקר - גם ככה מופיע כנקודת עניין היום כי הוא ממוזג עם נקודה של ויקיפדיה.

אשמח לעוד דוגמאות, אלא אם הטענות שלי לגבי מיפוי יותר מדויק מרגישות לכם חלשות.

zstadler commented 1 year ago

ראשית עלי לאמר שהנושא לא מאד חשוב בעייני ונשאבתי חזרה לדיון לבקשתך, @HarelM 😆

בכללי, אני מעדיף להיות טולרנטי לטעויות תיוג כמו חוסר של תגית תצפית/מונומט או העלאה של אותן תמונות לשתי נקודות OSM שלא ניתן למזג אותן. לדעתי, בהנתן כמות נקודות העניין שיש לנו, עוד 44 נקודות הן טיפה בים ועדיף שתהיינה שם.

HarelM commented 1 year ago

סבבה, אולי ככה זה יעלה את המודעות לנקודות ויותר אנשים ישפרו את המיפוי שלהן (במקום רק פסגה יוסיפו עוד תגיות, כמובן רק ע"י עורכים מתקדמים).

HarelM commented 1 year ago

This should be available now.