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
Added preset for boundary=marker #8113
base: master
Are you sure you want to change the base?
Conversation
Did you run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
Some changes are needed
actually you just need to run |
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
1fd90b3
to
869622c
Compare
Thank you for your comments. I tried to resolve them, please let me know if I missed something. |
The latest changes look good, thanks! Please rebase your PR onto the current master (right now your PR has two unrelated commits), then run Would you want to include more translations in this PR also? |
Actually one needs a Deepl api key to run it, so here is the result which you can integrate:
|
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
869622c
to
65d8a82
Compare
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
@organicmaps/translations review these translations please. |
I still have some troubles running
I updated translations manually. Tbh I am not sure if I will manage to fix All other comments should be resolved, thanks for patience |
@kubahaha please try to downgrade your protobuf version, e.g. I have 3.19.4 installed (via pip3) and it works well. upd: and what is your current version by the way? |
eu = Mugarria |
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
It was Should I update translations file according to comments? I changed manuall [pl] translation, as I am native speaker of polish. |
Awesome! |
data/strings/types_strings.txt
Outdated
pt = Marcador de fronteira | ||
pt-BR = Marcador de limite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pt = Marcador de fronteira | |
pt-BR = Marcador de limite | |
pt = Marco de fronteira |
PT = PT-BR = Marco de fronteira (since it's equal, no need to have PT-BR duplicated)
es it's OK. |
data/strings/types_strings.txt
Outdated
pl = Znak graniczny | ||
pt = Marcador de fronteira | ||
pt-BR = Marcador de limite | ||
ro = Marcaj de delimitare |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ro = Marcaj de delimitare | |
ro = Piatră de hotar |
en = Boundary Marker |
|
data/strings/types_strings.txt
Outdated
ca = Marcador de límit | ||
cs = Hraniční značka | ||
da = Grænsemarkør | ||
de = Grenzmarkierung |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should de = Grenzstein.
I think that some number of other translations are also not correct (for example probably Czech), maybe one could take the translations from Wikipedia: https://en.wikipedia.org/wiki/Boundary_marker
Signed-off-by: Kuba Mędrek <kubix.medrek@gmail.com>
Hello
I tried to add preset for boundary=marker. Can someone take a look please?
Following instructions from docs/STYLES.ms I was unable to run
tools/unix/generate_symbols.sh
and next scripts due to following CMake errors:can someone help me fix that dependencies, or by any chance would like to run it on already configured environment?