Improve fences and walls rendering #2063

Closed
opened 2022-02-09 18:43:18 +00:00 by pastk · 10 comments
Owner

At the moment fences, walls, hedges etc. are rendered as a very thin dotted line which is barely visible on the map, though its an important information for pedestrians and hikers.

Screenshot_1644352467

Let's make rendering more prominent and perhaps different for walls and fences (and maybe hedges too).

Samples:
fence https://www.openstreetmap.org/way/631353746
city_wall https://www.openstreetmap.org/way/129425886

Discussed in https://github.com/organicmaps/organicmaps/discussions/2046

At the moment fences, walls, hedges etc. are rendered as a very thin dotted line which is barely visible on the map, though its an important information for pedestrians and hikers. ![Screenshot_1644352467](https://user-images.githubusercontent.com/18434508/153071210-0e4f163a-339a-4135-ba0a-881d2820b2e8.png) Let's make rendering more prominent and perhaps different for walls and fences (and maybe hedges too). Samples: fence https://www.openstreetmap.org/way/631353746 city_wall https://www.openstreetmap.org/way/129425886 ### Discussed in https://github.com/organicmaps/organicmaps/discussions/2046
Author
Owner
The current rendering rules: https://github.com/organicmaps/organicmaps/blob/71de1db197343b8e02dada281327d99c74de3bdc/data/styles/clear/include/Basemap.mapcss#L670-L700
aesha-g commented 2022-03-18 04:01:42 +00:00 (Migrated from github.com)

@biodranik @rtsisyk
I want to solve this issue. can you please give a hint on where to start looking at this code? as I am new to development, I have successfully done the local setup.

thanks in advance.

@biodranik @rtsisyk I want to solve this issue. can you please give a hint on where to start looking at this code? as I am new to development, I have successfully done the local setup. thanks in advance.
biodranik commented 2022-03-18 06:36:52 +00:00 (Migrated from github.com)

Check drape rendering engine.

Check drape rendering engine.
Author
Owner

@aeshaganatra please see my comment above with a link to rendering rules.

Here is documentation on details how to change and test it.

If you have any questions - feel free to ask!

@aeshaganatra please see [my comment above](https://git.omaps.dev/organicmaps/organicmaps/issues/2063#issuecomment-1034082813) with a link to rendering rules. Here is [documentation](https://github.com/organicmaps/organicmaps/blob/master/docs/STYLES.md) on details how to change and test it. If you have any questions - feel free to ask!
aesha-g commented 2022-03-23 02:14:39 +00:00 (Migrated from github.com)

@biodranik @pastk
I'm trying to change above mention files. but while running the script "tools/unix/generate_drules.sh", I have encountered this error. This should not happen, because in mapCss StyleChooser is present. Have you ever occurred with this issue?

rm: cannot remove 'tools/unix/../../data/classificator.txt': No such file or directory
rm: cannot remove 'tools/unix/../../data/types.txt': No such file or directory
rm: cannot remove 'tools/unix/../../data/visibility.txt': No such file or directory
rm: cannot remove 'tools/unix/../../data/colors.txt': No such file or directory
rm: cannot remove 'tools/unix/../../data/patterns.txt': No such file or directory
Building drawing rules for style clear/clear
rm: cannot remove 'tools/unix/../../data/drules_proto_clear.bin': No such file or directory
rm: cannot remove 'tools/unix/../../data/drules_proto_clear.txt': No such file or directory
Traceback (most recent call last):
File "tools/unix/../../tools/kothic/src/libkomwm.py", line 1, in
from mapcss import MapCSS
File "/home/aesha/devlopment/organicmaps/tools/kothic/src/mapcss/init.py", line 21, in
from StyleChooser import StyleChooser
ModuleNotFoundError: No module named 'StyleChooser'

@biodranik @pastk I'm trying to change above mention files. but while running the script "tools/unix/generate_drules.sh", I have encountered this error. This should not happen, because in mapCss StyleChooser is present. Have you ever occurred with this issue? rm: cannot remove 'tools/unix/../../data/classificator.txt': No such file or directory rm: cannot remove 'tools/unix/../../data/types.txt': No such file or directory rm: cannot remove 'tools/unix/../../data/visibility.txt': No such file or directory rm: cannot remove 'tools/unix/../../data/colors.txt': No such file or directory rm: cannot remove 'tools/unix/../../data/patterns.txt': No such file or directory Building drawing rules for style clear/clear rm: cannot remove 'tools/unix/../../data/drules_proto_clear.bin': No such file or directory rm: cannot remove 'tools/unix/../../data/drules_proto_clear.txt': No such file or directory Traceback (most recent call last): File "tools/unix/../../tools/kothic/src/libkomwm.py", line 1, in <module> from mapcss import MapCSS File "/home/aesha/devlopment/organicmaps/tools/kothic/src/mapcss/__init__.py", line 21, in <module> from StyleChooser import StyleChooser ModuleNotFoundError: No module named 'StyleChooser'
biodranik commented 2022-03-23 11:24:36 +00:00 (Migrated from github.com)

Looks like a python issue

Looks like a python issue
Member

Have you ever occurred with this issue?

Make sure you've cloned the submodules, you can do this if you've already cloned the repo by running git submodule update --init

On the topic of fence rendering, organicmaps/organicmaps#2275 should also be considered, although it's much more complex

> Have you ever occurred with this issue? Make sure you've cloned the submodules, you can do this if you've already cloned the repo by running `git submodule update --init` On the topic of fence rendering, https://git.omaps.dev/organicmaps/organicmaps/issues/2275 should also be considered, although it's much more complex
aesha-g commented 2022-03-24 18:45:01 +00:00 (Migrated from github.com)

@endim8 which python version you're using?
I'm using 3.6.0.
I tried

git submodule update --init

but still not working for me.

@endim8 which python version you're using? I'm using 3.6.0. I tried > git submodule update --init but still not working for me.
biodranik commented 2022-03-24 19:46:02 +00:00 (Migrated from github.com)

@pastk we need to finish kothic's migration to python3.

@pastk we need to finish kothic's migration to python3.
Author
Owner

@endim8 which python version you're using? I'm using 3.6.0. I tried

git submodule update --init

but still not working for me.

Unfortunately scripts in organicmaps/tools/kothic/* require python 2.x still.
Do you have an opportunity to install python 2 alongside 3?

> @endim8 which python version you're using? I'm using 3.6.0. I tried > > > git submodule update --init > > but still not working for me. Unfortunately scripts in `organicmaps/tools/kothic/*` require python 2.x still. Do you have an opportunity to install python 2 alongside 3?
This repo is archived. You cannot comment on issues.
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
3 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#2063
No description provided.