search

bdunderscore / modular-avatar

Other
482 stars 65 forks source link

Menu Installerでインストールするメニューのアイコンのサイズが256より大きいときにエラーを吐くようにする #433

Closed anatawa12 closed 1 year ago

anatawa12 commented 1 year ago

VRCSDK 3.3.0でMA後にビルドチェックが通るようになりました。そのため、チェックをすり抜けていたメニューを使ってたアバターがエラーになるようになったようです。

エラーが"Images used for Actions & Moods are too large."だけで分かりづらいので、MAのMenu InstallerのValidate時に確認できるといいと思いました。

参考1: VRCSDK 3.0.9時点での該当チェックコード https://github.com/vrchat/packages/blob/25fcb7285e96eb0b1c10fd50d4daaed5d79e7a83/packages/com.vrchat.avatars/Editor/VRCSDK/SDK3A/VRCSdkControlPanelAvatarBuilder3A.cs#L86-L178

参考2: 実際のエラーになった例のツイート https://x.com/Shell_Beige/status/1701934893469745212?s=20 https://x.com/Shell_Beige/status/1701945289882628157?s=20

bdunderscore commented 1 year ago

https://github.com/bdunderscore/modular-avatar/issues/420 と関連性あり（いつの間にかこの辺のチェックがデッドコードになっていたことに、リファクタリング中に気付いた）

bdunderscore commented 1 year ago

対策としては縮小処理を入れたいと考えています