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: UVUsageCompabilityAPI for TTT AtlasTexture compatibility #1269

Merged
merged 5 commits into from
Oct 17, 2024

Conversation

anatawa12
Copy link
Owner

@anatawa12 anatawa12 commented Oct 16, 2024

Fix #1037

@anatawa12
Copy link
Owner Author

anatawa12 commented Oct 16, 2024

@ReinaS-64892 こんな感じの API で問題なさそうですかね

(一応テストして動いたつもり)

@ReinaS-64892
Copy link
Contributor

一応動いたけど... MeshRemovalProvider の方の対応とかなり相性が悪くてどうした物かって感じだね...一応それを無効化すると正しく動作するけども...
ReinaS-64892/TexTransTool#687

Island としての無効化をしながら、 Mask としての適用を維持できる状態というのがそもそも結構無理がある話な気がしてくる...
そもそも AtlasTexture のタイミングじゃなくて AAO の Removal を見る直前のタイミングで元のUVを伝えたほうがいい気もしてくるけど...そうなると UV1 に退避した時、ユーザー定義で UV1 に書き込まれたときに破綻するわけだからそれを事前に避けるのも難しい話...どうした物かな...

@ReinaS-64892
Copy link
Contributor

...一応 Removal を見るときに削除される適当な頂点のUV座標に寄せることでごまかしはできるが、 Box などのほかの削除系が混ざっているときにランダム頂点インデックスによっては破綻しうるから難しいね...

@anatawa12
Copy link
Owner Author

あー
MeshRemovalProviderがわでも退退避情報見るべきなのか。見るように変更しますね

@anatawa12
Copy link
Owner Author

MeshRemovalProviderで退避情報見るようにしました

@ReinaS-64892
Copy link
Contributor

UVの退避のタイミングを調整した実装で正しく動くことが確認できたよ!対応ありがとうございます!

@anatawa12
Copy link
Owner Author

じゃぁマージして良さそうかな

@anatawa12 anatawa12 marked this pull request as ready for review October 17, 2024 12:54
@anatawa12 anatawa12 merged commit ed5c538 into master Oct 17, 2024
9 checks passed
@anatawa12 anatawa12 deleted the tex-trans-tool-compability branch October 17, 2024 13:00
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.

Remove Mesh by MaskがTTTと互換性がない
2 participants