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

engineKeyをengineIdにすべて変更する #871

Merged
merged 5 commits into from
Aug 2, 2022

Conversation

Hiroshiba
Copy link
Member

内容

複数エンジン対応のために、engineIdとengineKeyという概念が作られていました。

engineIdはエンジンが固有で持つID、engineKeyはソフトがエンジンごとに勝手に割り振るIDです。
同じengineIdでも、バージョンが違えば別のkeyを割り当てることができるように設計されています。

現状全部engineKeyになっていますが、こちらのPRでengineIdが導入される予定でしたが、まだengineKeyとengineIdの使い分けを議論しきれていない状況です。

そこで一度、VOICEVOXエディタはエンジンの種類ごとに1つだけしかエンジンを把握できない、という設計にしたいと思います。
つまり、engineKeyをなくして、engineIdのみにする戦略です。

engineKeyはあとで導入することもできるはずです。
engineKeyを活かすには更にUIの実装も必要なため、実装をいったん後回しにしたいと思います。

関連 Issue

その他

この変更により、.envのkeyをidに変えないと起動しなくなります。
あとでissueで案内したいと思います。

@Hiroshiba
Copy link
Member Author

@aoirint さん、 @Segu-g さん、よかったらレビュー(+意見やコメントもあれば)ぜひお願いします・・・!

Copy link
Member

@Segu-g Segu-g left a comment

Choose a reason for hiding this comment

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

このPRの目的としては、異なるengineIdのエンジンしか動かせないようにするといった事でしょうか?
このブランチだとキャラクターによってエンジンを選択する操作が未実装なのでなんとも言えませんが、UIについて考えないのならengineIdよりもengineKeyで管理しておいた方がユースケースは広そうです

.env.production Outdated Show resolved Hide resolved
@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jul 24, 2022

このPRの目的としては、異なるengineIdのエンジンしか動かせないようにするといった事でしょうか?

目的は単純に、keyという概念を後から刺しやすいように、idに置き換えたい感じです。

将来のことを考えると、keyかidどちらにするかで影響があるのは、保存項目だと思います。
例えばプロジェクトファイルに保存されるAudioItemとかです。
AudioItemは一意な音声を生成するための情報を持っていてほしいはずです。
となると、engineIdとengineVersion(もしくはlibraryVersionやcoreVersion)を保存する必要が出てくると思います。
なので今のうちにengineKeyではなく、engineIdとして扱って保存するようにしておこうかなという感じです。
(この理由はさっき思いつきました・・・。)

@Hiroshiba
Copy link
Member Author

id->uuidに変更しました!

@Hiroshiba
Copy link
Member Author

マージします!

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