Rename --gui option to --cui option etc

axinc-ai / ailia-models

The collection of pre-trained, state-of-the-art AI models for ailia SDK

2.04k stars 325 forks source link

Closed ohshima-axell closed 2 months ago

ohshima-axell commented 2 months ago

既存のモデルの --gui オプションを --cui オプションといった名前に変更し、GUI 環境でウィンドウが出る挙動の方をデフォルトにするようにしました。また、lipgan に GUI がない環境用のコードが入っていなかったので --cui オプションを合わせて追加しました。

以下は古い description

lipgan が GUI のない環境でエラーとなるため、siam-mot を参考に --gui オプションを追加して GUI のない環境でも実行できるようにしました。

kyakuno commented 2 months ago

@ohshima-axell ユーザがデフォルトで期待する動作はGUIを表示することな気がしまして、--cuiオプションにして、デフォルトは表示する方向ではいかがでしょうか？

ohshima-axell commented 2 months ago

@kyakuno その方針でも良いと思います。 --gui オプションがあるモデルは他にもあるので、本 pull request の内容を以下ようにしようと思いますが問題ないでしょうか？

kyakuno commented 2 months ago

動画を表示する方に寄せて、検証用に--cuiを作る方針でお願いします。

ohshima-axell commented 2 months ago

@kyakuno --gui オプションの置き換え作業を行いました。確認よろしくお願いします。

kyakuno commented 2 months ago

@ohshima-axell 修正、ありがとうございます。segment-anythingだけ、guiのままにして頂いても良いでしょうか？segment-anythingは通常はcuiで動かしていて、特殊な用途でのみguiを使う形になります。

kyakuno commented 2 months ago

対応ありがとうございました！