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.

Sign up for GitHub

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

Jump to bottom

Display water objects: rapids and reef, dam and more #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Display water objects: rapids and reef, dam and more #6

wants to merge 3 commits into from

Conversation

noktoborus
Copy link
Contributor

No description provided.

@noktoborus noktoborus changed the title Display rapids on river surface Display water objects: rapids and reef Mar 15, 2023
@noktoborus noktoborus force-pushed the add_rapids branch 2 times, most recently from 75cab1d to e55b761 Compare March 17, 2023 12:37
@noktoborus noktoborus changed the title Display water objects: rapids and reef Display water objects: rapids and reef, dam and more Mar 17, 2023
molind
molind requested changes Apr 27, 2023
View reviewed changes
Copy link
Collaborator

@molind molind left a comment

Choose a reason for hiding this comment

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

Я бы предложил просто черным рисовать дамбы. кружок, линия, полигон (черная обводка и серая заливка) как в osm wiki показывают: https://wiki.openstreetmap.org/wiki/Tag:waterway%3Ddam
Серым водосливы. кружок, серый пунктир, полигонов вроде там нет.
рифы и рапиды вроде норм. в темной теме надо их перепроверить.

@noktoborus
Copy link
Contributor Author

noktoborus commented Apr 30, 2023 •
edited
Loading

Я бы предложил просто черным рисовать дамбы. кружок, линия, полигон (черная обводка и серая заливка) как в osm wiki показывают: https://wiki.openstreetmap.org/wiki/Tag:waterway%3Ddam Серым водосливы. кружок, серый пунктир, полигонов вроде там нет. рифы и рапиды вроде норм. в темной теме надо их перепроверить.

Хорошо, поправлю как доберусь до этих ревью :)

@noktoborus
Copy link
Contributor Author

Screenshot_20230506_002513_Guru Maps Pro
Screenshot_20230506_002526_Guru Maps Pro
Screenshot_20230506_002810_Guru Maps Pro
Screenshot_20230506_002800_Guru Maps Pro

@noktoborus
Copy link
Contributor Author

Я бы предложил просто черным рисовать дамбы. кружок, линия, полигон (черная обводка и серая заливка)

использовал цвета синих оттенков, посмотри, хорошо ли.

Серым водосливы. кружок, серый пунктир, полигонов вроде там нет.

Водосливы это weir? Полигоны на wiki osm не рекомендуются, но в БД их 65 тыщ. Судя по weir на рощинке (последние два скриншота) area waterway=weir отмечают неправильные объекты.

@noktoborus
Copy link
Contributor Author

Screenshot_20230506_003743_Guru Maps Pro
отображение area rapids сломалась (на geojson было норм), проверю потом. Пока мержить не нужно.

@noktoborus
Copy link
Contributor Author

noktoborus commented May 18, 2023 •
edited
Loading

reef:

Screenshot_20230518_163118_Guru Maps Pro
Screenshot_20230518_163111_Guru Maps Pro

dam:

Screenshot_20230518_140103_Guru Maps Pro
Screenshot_20230518_140056_Guru Maps Pro

@noktoborus
Copy link
Contributor Author

rapids удалил: выглядит так, будто fill-image исключает fill-color
reef перерисовал в svg, выглядит посимпатичнее и понятнее точек

@noktoborus noktoborus requested a review from molind May 18, 2023 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@molind molind Awaiting requested review from molind

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@noktoborus @molind