Skip to content

[lemkotools.rue.lemko] Add Lemko predictive-text model - #354

Merged
darcywong00 merged 4 commits into
keymanapp:masterfrom
paulolemko:lemko-model-0.3.0
Oct 26, 2026
Merged

darcywong00 merged 4 commits into
keymanapp:masterfrom
paulolemko:lemko-model-0.3.0

Conversation

@paulolemko

@paulolemko paulolemko commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Adds a Lemko predictive-text model containing 219,273 distinct forms, associated with language rue and intended for use with the keyboard in keymanapp/keyboards#4191.

Frequencies for 69,465 forms were measured in 1,766 texts after removing exact duplicates, reconstructed from a subset of an indexed corpus. Normalization preserves distinctions between і/ї, и/й and г/ґ.

@keyman-server

Copy link
Copy Markdown

Thank you for your pull request. You'll see a "build failed" message until the Keyman team has reviewed the pull request and manually initiated the build process.

Every change committed to this branch will become part of this pull request. When you have finished submitting files and are ready for the Keyman team to review this pull request, please post a "Ready for review" comment.

@paulolemko

Copy link
Copy Markdown
Contributor Author

Ready for review

@paulolemko
paulolemko marked this pull request as ready for review October 16, 2026 20:40
@paulolemko

Copy link
Copy Markdown
Contributor Author

Hello, could you please review this lexical model submission and rerun the build? The “Test: Pull Requests (Models)” check has been failing since October 16. I downloaded the current PR files and successfully compiled both the model and its KMP package locally with kmc 18.0.253.

I couldn’t access the TeamCity build log. If the build fails again, could you please share the error details so I can address them?

The companion keyboard has now been merged and published. Thank you for your help!

@darcywong00

Copy link
Copy Markdown
Contributor

Sorry for the delay. We're a little short-staffed on availability to review PR's.

I've manually triggered a Team City build so we should see if it passes later today...

@darcywong00 darcywong00 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm - thanks for the submission!

@paulolemko

Copy link
Copy Markdown
Contributor Author

Thank you for reviewing the submission, Darcy! I’ve removed the .model_info file as requested. The update is ready for review.

@darcywong00
darcywong00 merged commit e3918e6 into keymanapp:master Oct 26, 2026
2 checks passed
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