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

辞書のインポート/エクスポートGUIを提供する #656

Closed
Hiroshiba opened this issue Apr 6, 2023 · 9 comments
Closed

辞書のインポート/エクスポートGUIを提供する #656

Hiroshiba opened this issue Apr 6, 2023 · 9 comments

Comments

@Hiroshiba
Copy link
Member

内容

辞書のインポート・エクスポート機能は結構需要があります。
https://twitter.com/nicogorivv/status/1603656354249392128
https://twitter.com/Inaba_Mkn/status/1643944206002712576

APIはあるけどまだGUIが無く、もし作ったらユーザーからのニーズに答えられそうです。

(実はエンジン側にはGUI機能があります)

Pros 良くなる点

PC移行などに便利

Cons 悪くなる点

javascriptを書く必要がある?

実現方法

APIを叩くコードをui_template/ui.htmlに書く?

その他

@Hiroshiba
Copy link
Member Author

@My-MC さんとかご興味ありませんか・・・? 👀

@My-MC
Copy link
Contributor

My-MC commented Apr 6, 2023

ちょうど時間もありますのでぜひ取り掛からせて頂けますか。

@Hiroshiba
Copy link
Member Author

おーーー!!ぜひ・・・!!!

@My-MC
Copy link
Contributor

My-MC commented Apr 22, 2023

ご相談失礼します。
いまアップロード、ダウンロードの機構が完成しているのですが、インポートを実装するうえで完全上書きにするか、pydanticを使って差分保存をするか、どちらが良いでしょうか。

@Hiroshiba
Copy link
Member Author

なるほどです、大切な観点だなと思いました!

ちょっと両者の利点欠点が僕の中で掴めてないかもです。
完全上書きだと簡単だけど既存のが消えてしまう、という感じでしょうか?

@My-MC
Copy link
Contributor

My-MC commented Apr 22, 2023

はい完全上書きは、完全にファイル丸ごと置き換えるので既存のものが消えてしまい、pydanticの差分保存だと、pydanticのBaseModelを使って差分を追加することができます。参考です。FastAPIのチュートリアル

@Hiroshiba
Copy link
Member Author

なるほどです!! 差分便利ですね。

今確認してみたのですが、run.pyにあるimport_user_dictを使うか、その中で利用されているimport_user_dictを使えばそもそも差分読み込みかもです・・・!

@app.post("/import_user_dict", status_code=204, tags=["ユーザー辞書"])

@tarepan
Copy link
Contributor

tarepan commented Dec 11, 2023

Latest release バージョン GUI を用いて検証をおこないました。
エクスポート/編集/インポートで正常に辞書が更新されたことを確認しました。
image

@My-MC さん good work!

@Hiroshiba
本 issue は目標達成につき close 可能です。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Dec 11, 2023

おお、本当ですね! ご確認ありがとうございます、closeさせていただきます!!
@My-MC さんありがとうございました!!
(ちなみに本機能はエンジン0.15でリリース予定です!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants