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

translations adapted #1976

Open
wants to merge 37 commits into
base: dev
Choose a base branch
from
Open

translations adapted #1976

wants to merge 37 commits into from

Conversation

Staubgeborener
Copy link

  • Please check if the PR fulfills these requirements
  • The changes have been tested locally
  • There are no breaking changes
  • What kind of change does this PR introduce?
  • correct indention in all translations
  • add "displayText": "HallSensor\nSleepTime" for for many translations
  • What is the current behavior?
  • wrong indentions in all translations in specific are around "displayText": "HallSensor\nSleepTime"
  • missing "displayText": "HallSensor\nSleepTime" translation
  • What is the new behavior (if this is a feature change)?
    see What kind of change does this PR introduce?

  • Other information:

auto-merge was automatically disabled September 20, 2024 06:55

Head branch was pushed to by a user without write access

auto-merge was automatically disabled September 20, 2024 08:22

Head branch was pushed to by a user without write access

@discip
Copy link
Collaborator

discip commented Oct 4, 2024

@Staubgeborener
Sorry, I somehow lost track of this.
Do you have any idea what the problem could be other than your recent attempts to fix the compilation errors?
I tried to look for a possible cause, but unfortunately I have absolutely no clue what to blame.

@Ralim
Maybe you are able to spot the cause. 😊

@Staubgeborener
Copy link
Author

@Staubgeborener Sorry, I somehow lost track of this. Do you have any idea what the problem could be other than your recent attempts to fix the compilation errors? I tried to look for a possible cause, but unfortunately I have absolutely no clue what to blame.

@Ralim Maybe you are able to spot the cause. 😊

  File "/__w/IronOS/IronOS/source/../Translations/make_translation.py", line 532, in get_font_map_per_font
    raise KeyError(f"Symbol |{sym}| is missing in large font set")
KeyError: 'Symbol |ợ| is missing in large font set'
make: *** [Makefile:548: Core/Gen/Translation.VI.cpp] Error 1
make: *** Waiting for unfinished jobs....
Error: Process completed with exit code 2.

It seems that certain special characters that occur in some languages cannot be interpreted correctly (maybe this function).

@Ralim
Copy link
Owner

Ralim commented Oct 6, 2024

Hia,

KeyError: 'Symbol |ợ| is missing in large font set'
This is thrown deliberately if we do not have that character in our font table.
This is to prevent Symbols we dont have being used (unless they are added).
Symbols are defined here https://github.com/Ralim/IronOS/blob/fe6bf7f1f04d2d958a2acb66eeb0bca4701d6fc1/Translations/font_tables.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants