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

feat: add TextureCompressor to compress textures on serialization #228

Closed
wants to merge 1 commit into from

Conversation

kurotu
Copy link
Contributor

@kurotu kurotu commented Apr 26, 2024

resolve #225 and #164

BuildContext.Serialize() 内で未保存のテクスチャアセットがある場合に、保存前に自動的に圧縮します。
また最後にテクスチャのフォーマットを確認し、現在のプラットフォームで非サポートの場合は警告を表示します。

image

@anatawa12 @ReinaS-64892
関係すると思うので見てもらえると助かります。

@bdunderscore
Copy link
Owner

これについては、ビルド終了時特定のアセットへの参照がなかった場合のみコールバックを走らせる仕組みを実装する予定ですが、それでは足りない感じでしょうか?

@kurotu
Copy link
Contributor Author

kurotu commented Apr 27, 2024

保存時にコールバックできるならそれでも十分だと思います。むしろ、そちらの方がテクスチャ以外にも応用が利きますし、NDMFの責務をあまり増やさずに済んで良さそうです。
このPR、NDMFでメンテナンスしなきゃいけない部分が増えるのが気になっていたので。

@bdunderscore
Copy link
Owner

#229 のほうで対応するので、こちらの方はクローズさせていただきます。

@kurotu kurotu deleted the texture-compressor branch April 28, 2024 02:54
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