geonlp-platform / pygeonlp

pygeonlp, A python module for geotagging Japanese texts.
https://geonlp.ex.nii.ac.jp/
BSD 2-Clause "Simplified" License
19 stars 1 forks source link

メモリリークしている件について #19

Open kinsho-stb opened 8 months ago

kinsho-stb commented 8 months ago

お世話になっております。

pygeonlpを利用しており、プロファイリングの結果から以下の2メソッドがメモリリークしていそうです。

こちらご確認いただけないでしょうか、よろしくお願いいたします。

※ c++の2つの対象関数呼び出し元のpythonコード上関数名がグラフに表示されております image

shigeki-takahashi-stb commented 5 months ago

お世話になっております。

こちらメモリリークの件についてdevelopブランチにはマージ済のようですが、 もしよろしければ正式リリースの予定が決まっていましたらご教示いただけますでしょうか。

KatHaruto commented 5 months ago

@shigeki-takahashi-stb メンションつけないと気づかれないかもしれません。。

@geonlp-platform @t-sagara 上のコメントいかがでしょうか

t-sagara commented 5 months ago

@shigeki-takahashi-stb @KatHaruto ご指摘ありがとうございます。 バージョンを v1.2.2.post1 として main ブランチにマージし、 PyPI に登録しました。