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

Create ISOM 2017-2v5_15000.xmap #2113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mlerjen
Copy link
Contributor

@mlerjen mlerjen commented Dec 5, 2022

Also here. New symbol set according to the color table and the most recent revision.

Also here. New symbol set according to the color table and the most recent revision.
@MaBreaker
Copy link

It seems that this merge does not just implement "minor" corrections to the existing symbol set, but introduces totally different approach to define symbols for multiple types of terrain objects like cultivated land, orchard etc. also many symbol sub numbers are just switched to another meaning compared to original symbol set provided within OOM.

This means that if anyone tries to do "update the latest symbol set" to the existing map created by using original symbol set (previous way of working) it would "broke" the whole map even if the best possible "crt/replacement" mapping is used.

Wouldn't it be better to try to keep "existing" sub numbering and also layout of the fields etc. and just "fix" symbol dimensions and colours there. That would make map updates and also github "change" tracking much easier.

@dg0yt
Copy link
Member

dg0yt commented Jun 19, 2024

For review: Report on colors and symbols

Bildschirmfoto vom 2024-06-19 08-05-50

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.

3 participants