Skip to content

[WIP] kab-Latn: add missing - #252

Closed
athmanemokraoui wants to merge 8 commits into
dmort27:masterfrom
athmanemokraoui:master
Closed

[WIP] kab-Latn: add missing#252
athmanemokraoui wants to merge 8 commits into
dmort27:masterfrom
athmanemokraoui:master

Conversation

@athmanemokraoui

@athmanemokraoui athmanemokraoui commented Jun 9, 2026

Copy link
Copy Markdown

bug fix :

  • Added missing geminates: tt, qq, jj, cc, čč, ǧǧ, hh, ww, yy, ḥḥ, ṣṣ, ṭṭ, ḍḍ, ẓẓ, ṛṛ
  • Mapped o to u since Kabyle has no native /o/ phoneme; words borrowed with /o/ are nativized as /u/
  • Removed v and vv since /v/ is not a native Kabyle phoneme and only appears in recent loanwords only

Sources of information for the rules

I'm a native kabyle speaker working of projects related to kabyle language.

Once this done, I'll work to introduced a pre and post processing for kabyle and to handle complex forms.

- Added missing geminates: tt, qq, jj, cc, čč, ǧǧ, hh, ww, yy, ḥḥ, ṣṣ, ṭṭ, ḍḍ, ẓẓ, ṛṛ
- Mapped `o` to `u` since Kabyle has no native /o/ phoneme; words borrowed with /o/ are nativized as /u/
- Removed v and vv since /v/ is not a native Kabyle phoneme and only appears in recent loanwords only
@athmanemokraoui

Copy link
Copy Markdown
Author

Hum, I think that the o should be removed too as it is not a standard in kabyle.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I disagree with removing v and vv since the goal of Epitran is not to adjudicate what graphemes do and do not belong in a language, but to be maximally robust to what might actually occur in data (and having these in the mapping should not negatively impact the performance of the module on native Kabyle).

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah okay! Good to know :)

r,r
s,s
t,t
v,v

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add back

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added back v at the end on the file.

@juice500ml

Copy link
Copy Markdown
Collaborator

FYI, current test for Kabyle is failing:

FAIL: test_basic_words (test_kabyle.TestKabyle.test_basic_words)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/epitran/epitran/epitran/test/test_kabyle.py", line 17, in test_basic_words
    self.assertEqual(tr, o)
    ~~~~~~~~~~~~~~~~^^^^^^^
AssertionError: 'θaqʃiʃt' != 'taqʃiʃt'
- θaqʃiʃt
? ^
+ taqʃiʃt
? ^

@athmanemokraoui athmanemokraoui changed the title kab-Latn: add missing geminates, map o to u, remove v/vv (non-native) kab-Latn: add missing geminates, map o to u Jun 16, 2026
@athmanemokraoui

athmanemokraoui commented Jun 16, 2026

Copy link
Copy Markdown
Author

FYI, current test for Kabyle is failing:

FAIL: test_basic_words (test_kabyle.TestKabyle.test_basic_words)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/epitran/epitran/epitran/test/test_kabyle.py", line 17, in test_basic_words
    self.assertEqual(tr, o)
    ~~~~~~~~~~~~~~~~^^^^^^^
AssertionError: 'θaqʃiʃt' != 'taqʃiʃt'
- θaqʃiʃt
? ^
+ taqʃiʃt
? ^

Nice one ! t case.
In the case of taqcict we should get θaqʃiʃθ actually better θæqʃiʃθ :) I'm testing.

Correcting kabyle test.

The word `tuwiḍ` has no meaning. Should use `tewwiḍ` (as in : you took with you). 

`ɣ` to `ʁ`
`b` to `β` in this specific case of `abrid` (road, street).
Adding `æ`
Adding `ə`
@athmanemokraoui athmanemokraoui changed the title kab-Latn: add missing geminates, map o to u [WIP] kab-Latn: add missing Jun 16, 2026
@athmanemokraoui

athmanemokraoui commented Jun 16, 2026

Copy link
Copy Markdown
Author

May be I should close this as I'm working on it to correct some things?

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