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

Use tokenize instead of getTokenizer #16

Merged
merged 2 commits into from
Dec 12, 2021
Merged

Conversation

k-tahiro
Copy link
Contributor

ルール間で形態素解析結果を使いまわせるようにtokenizeを使用するPRです。

@azu
Copy link
Member

azu commented Dec 10, 2021

https://github.com/azu/kuromojin/blob/374bba24329507caa16c2e5b1cdc9e69c1062a76/src/kuromojin.ts#L75-L99
kuromojinの getTokenizer の方にもLRUキャッシュが効くように変更しちゃうのはどうですかね?
getTokenizer でキャッシュ入れてないのはあんまり一貫性がない気がするので、入れてしまうのが楽なのかなーと思って。

@k-tahiro
Copy link
Contributor Author

kuromoji.jsのTokenizerではなくて、kuromojinのTokenizerクラスを作って返す感じですかね。
kuromojinのgetTokenizerがどういった使われ方を想定しているのかを詳しく知っているわけではないですが、良いんじゃないかなと思います。

@azu
Copy link
Member

azu commented Dec 12, 2021

kuromoji.jsのTokenizerではなくて、kuromojinのTokenizerクラスを作って返す感じですかね。

はい。

https://github.com/azu/kuromojin/blob/374bba24329507caa16c2e5b1cdc9e69c1062a76/src/kuromojin.ts#L9-L12
型的には2つのメソッド持つラッパーになってるので、ラッパー的なオブジェクトを返すとかで良いのかなーと思いました。

@azu azu merged commit 65dc1a4 into textlint-ja:master Dec 12, 2021
@azu
Copy link
Member

azu commented Dec 12, 2021

別途Issueを作りました
azu/kuromojin#13

これ自体は問題ないため、マージしてしまいます。

@azu
Copy link
Member

azu commented Dec 12, 2021

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.

2 participants