search

ArduPilot / MethodicConfigurator

A clear ArduPilot configuration sequence
https://ardupilot.github.io/MethodicConfigurator/TUNING_GUIDE_ArduCopter
GNU General Public License v3.0
44 stars 14 forks source link

i18n markup added #48

Closed BloodSakura3774 closed 3 weeks ago

BloodSakura3774 commented 2 months ago

I used regular expressions to do the markup. Both logging and output/display strings are marked. I'm not sure about some of them so please double check my work before merging. And I'm not certain about some of the formated, please notice me if the format isn't right and I'll fix them. I might added too much or missed some.

amilcarlucas commented 2 months ago

Does the code still run? Have you tested it?

BloodSakura3774 commented 2 months ago

Does the code still run? Have you tested it?

Didn’t test it, I don’t think the code will run without implementing the library first. Will test later

BloodSakura3774 commented 2 months ago

Does the code still run? Have you tested it?

It won't run without gettext is implemented, Syntax error in ...\MethodicConfigurator\ardupilot_methodic_configurator.py File "...\MethodicConfigurator\ardupilot_methodicconfigurator.py", line 50 ('parameters. The GUI reads intermediate parameter files from a directory and ') ^ SyntaxError: invalid syntax

amilcarlucas commented 2 months ago

This week I'm really busy can you add commits that start the gettext implementation?

BloodSakura3774 commented 2 months ago

I can try to finish this

BloodSakura3774 commented 1 month ago

implemented, and tested that it works. only thing left is exporting those marked string and translate them. Also it is currently set to zh-cn in the new file internationalization.py I believe we will need to compile multiple .exe for release after this is merged. *looks like the folder locale is not present, please add one yourself.

amilcarlucas commented 1 month ago

My plan is to allow the user to set the language during installation, and only distribute a single .exe installer with all the languages. I corrected some stuff, and removed the translation of some of the auxiliary files. Those can be added later as part of another PR. For now we should only foucus on the file on this rebased and squashed commit.

BloodSakura3774 commented 1 month ago

Yes it will be better but there is more work to do by distributing a single .exe. TBH, some chinese users can not read any english so setting the language when installing might be "difficult" for them. If we are doing only chinese translation for now, I think it's better to distribute two .exe. I'm available during the weekend so can you start extrating those marked strings?

amilcarlucas commented 1 month ago

This branch produces a chinese .exe file. The .po file is ready for translation and there is a script to generate the .mo file.

Can you take it from here and add the translation? I will add the language selection at the beginning of the installer later.

amilcarlucas commented 3 weeks ago

An improved version of this got merged, thanks for helping out. See details in https://github.com/ArduPilot/MethodicConfigurator/commit/3ca5a385b34612e9949a94cd5d6193e51c458a49

Please open a new pull-request on top of master

BloodSakura3774 commented 3 weeks ago

I'll try finishing it this weekend. There is more string than I thought would be. I'm trying to make it as natural as possible.