-
Notifications
You must be signed in to change notification settings - Fork 20
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
Test failure with Korean #38
Comments
On Fedora, the Korean hunspell dictionary was recently updated and therefore I had to adapt the test. |
See: |
That change was already in ibus-typing-booster 1.5.37 though. Did the test case still |
The comment at the start of the Korean test case says that there is no Korean hunspell dictionary def test_korean(self): Did you recently get a Korean hunspell dictionary on FreeBSD? |
And, the testcase ues the 'ko-romaja' input method. Do you have it available? On Fedora $ rpm -qf /usr/share/m17n/ko-romaja.mim |
Thanks for all these ideas!
I shall keep investigating and let you know. |
If you add this test case to m17n_translit.py:
Does this work? You can test like this:
If you see no output from “python3 m17n_translit.py”, then it works. |
I have upgraded libofx, m17n-db and m17n-lib to be sure that it's not the source of the problem, and the examples in m17n_translit.py fail, not only the Korean one. It produces NULL pointer access, see the attached log. |
Did you find anything? Does ibus-m17n work for you? |
(Sorry for the delay...)
|
Apparently you are using ibus-setup to look for the m17n input methods. That is OK
lists all engines ibus offers (same list you see in ibus-setup). But you can easily grep in the output.
shows that ibus offers 163 engines from ibus-m17n. Korean is not among them:
The reason is that the m17n:ko:* engines are not considered useful as there is also ibus-hangul specialized for Korean:
These engines which rank lower than 0 are omitted on purpose because something But m17n input methods which are not considered useful to offer via ibus-m17n might |
You write: “ibus-m17n seems working, at least for the languages that I can understand.” That makes it a bit mysterious why the examples in m17n_translit.py fail. Could you find out more why these examples are failing? |
I checked on freebsd now and not all tests in m17n-translit.py fail for me. Actually only those using inscript2 and the Korean one fail. That the inscript2 tests fail is because you don’t have the inscript2 input methods for m17n
And then:
After doing that, only the Korean test still fails:
|
That failure of the Korean test on FreeBSD is quite weird: Translating the hex codes into the real characters, this is: Expected: 안녕하세요 |
I added "Korean (romaja (m17n))" in the gnome-control-center, selected this The result is the same as in the m17n_translit test case, i.e. one gets 안녕세ㅛ instead of 안녕하세요 This seems clearly wrong, but as it is the same error when using ibus-m17n and |
I ignored the error in the Korean test case for the moment and tried instead whether I can I had to do two small fixes to make it work: commit cde1d57ad70158b1e01f1a681f5d863a39bc7379
commit eeeb2a7
This will be in the 2.0.1 release. |
Many points!
Do I miss some configuration step?
Many thanks for your time and all these ideas. I was too busy to check it today, but I shall work on it again ASAP! |
You write: “On my desktop, ibus list-engine only lists lines beginning with "xkb:", then ibus list-engine | grep m17n is empty.” I just did “pkg install m17n-lib”, “pkg install m17n-db” and “pkg install ibus-m17n”. Then restarted the gnome session to restart ibus. (“ibus restart” should also work and then there is no need to restart the Gnome session. But |
You write: “inscript2 I don’t know of any other home page than: https://releases.pagure.org/inscript2/ That work was done by Red Hat. I pinged the guy who did that today and asked him to In my ibus-typing-booster package for openSUSE, I just include that inscript2-20160423.tar.gz Of course ibus-typing-booster still works without that, but you can use only inscript and not inscript2 then for Indian languages. |
You wrote: “Did you install m17n from the ports or the packages? ATM they install version 1.7.0.” I used “pkg install m17n-lib”, “pkg install m17n-db”, and “pkg install ibus-m17n”. ibus-m17n-1.3.4.16 |
What you write about the Korean settings in KDE looks like keyboard settings, it seems This has nothing to do with the "ko-romaja" input method from m17n. |
Oops... I just noticed that after upgrading m17n-lib and m17n-db, I forgot to reinstall ibus-m17n! |
Could you find anything which causes the problem? |
Yes, it is surely specific to FreeBSD. |
The Korean test is the only one which is failing when doing
? |
Its output is:
|
Did you find anything new here? |
I've just upgraded ITB to 2.3.1, and launched the tests again. Meanwhile, many dependencies (ibus, etc.) have been upgraded.
|
I just upgraded to 2.6.5, and the above error with Korean is still there, but there is also one more failure:
|
Thierry Thomas <[email protected]> さんは書きました:
I just upgraded to 2.6.5, and the above error with Korean is still there, but there is also one more failure:
```
======================================================================
FAIL: test_accent_insensitive_matching_french_dictionary (test_itb.ItbTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/ports/textproc/ibus-typing-booster/work/ibus-typing-booster-2.6.5/tests/test_itb.py", line 577, in test_accent_insensitive_matching_french_dictionary
'différemment')
AssertionError: 'différemment po:adv' != 'différemment'
- différemment po:adv
? -------
+ différemment
```
grep po:adv /usr/share/myspell/fr_FR.dic
finds nothing in the French hunspell dictionary on my Fedora 30 system.
Maybe you have a different version of the French dictionary? Does your
dictionary contain a line "différemment po:adv"?
…--
📧 Mike FABIAN <[email protected]>
睡眠不足はいい仕事の敵だ。
|
OK, this is the reason: I don't have myspell, but hunspell, and
|
That looks like a bug the fr_FR.dic to me. Because each line in such a dictionary should |
This is a part of hunspell's specification: see p. 9 and 10 of https://grammalecte.net/_misc/hunspell4.pdf |
You are right, I’ll make the following change in the 2.6.6 release
|
Newer French dictionaries downloaded from http://grammalecte.net/download/fr/hunspell-french-dictionaries-v6.4.1.zip contain spaces to separate words from extra information. Improve the regexp to get the words correctly from these dictioniaries as well. These new French dictionaries seem to be available already on FreeBSD, see: #38 (comment)
The fix for the new French dictionaries is included here: https://github.com/mike-fabian/ibus-typing-booster/releases/tag/2.6.6 |
On FreeBSD, the test suite fails with this message:
Any idea?
The text was updated successfully, but these errors were encountered: