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

Rename BlendShape #1245

Merged
merged 47 commits into from
Oct 12, 2024
Merged

Rename BlendShape #1245

merged 47 commits into from
Oct 12, 2024

Conversation

anatawa12
Copy link
Owner

Fix #9

@anatawa12 anatawa12 marked this pull request as ready for review October 9, 2024 04:03
@anatawa12 anatawa12 added the enhancement New feature or request label Oct 9, 2024
@Sayamame-beans
Copy link
Sponsor Collaborator

とりあえず日本語更新しました

@anatawa12
Copy link
Owner Author

AAO Rename BlendShapeってModifying Edit Skinned Mesh Componentです? (記載なし)

はい。忘れですね

@anatawa12
Copy link
Owner Author

The cause of SIGSEGV is loooooong type name and the fact it's used in class name of delayCall.

PrefabSafeUniqueCollection<MapEntry<TKey, TValue>, TKey, PrefabSafeMapManipulator<TKey, TValue>> with string key/value would become Anatawa12.AvatarOptimizer.PrefabSafeUniqueCollection.PrefabSafeUniqueCollection`3[[Anatawa12.AvatarOptimizer.PrefabSafeMap.MapEntry`2[[System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089],[System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089]], com.anatawa12.avatar-optimizer.internal.prefab-safe-set, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null],[System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089],[Anatawa12.AvatarOptimizer.PrefabSafeMap.PrefabSafeMapManipulator`2[[System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089],[System.String, mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089]], com.anatawa12.avatar-optimizer.internal.prefab-safe-set, Version=0.0.0.0, Culture=neutral, PublicKeyToken=null]], and this would cause buffer overflow

@anatawa12
Copy link
Owner Author

CI通ったね...
F**k Unity C++

@anatawa12
Copy link
Owner Author

anatawa12 commented Oct 10, 2024

多分機能的にはOKなのでローカライズ/ドキュメント周りが良くなったらマージ(テストは書くかも

@Sayamame-beans
Copy link
Sponsor Collaborator

Edit Skinned Mesh Component側のページに記載忘れてる

@anatawa12
Copy link
Owner Author

Edit Skinned Mesh Component側のページに記載忘れてる

直しました。

@Sayamame-beans
Copy link
Sponsor Collaborator

多分良い感じになったと思う
changelogでmerge機能については言及しなくて良い?

@anatawa12
Copy link
Owner Author

anatawa12 commented Oct 12, 2024

マージはとりあえずおまけなので。

あとマージはこのまま行くかは少し検討したい

BlendShape結合による軽量化をやりたいって考えたらもっと高度なコンポーネントが欲しくなる気がするので、1.8リリース前にRename BlendShapeから消してv1.9で別コンポーネントを作るってやるかも

@anatawa12
Copy link
Owner Author

じゃぁマージします

@Sayamame-beans
Copy link
Sponsor Collaborator

👍

@anatawa12
Copy link
Owner Author

マージ仕様の是非について => #1250

@anatawa12
Copy link
Owner Author

SayamameさんがRequest Changesしたままだからマージできないわ

Copy link
Sponsor Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

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

good to go

@anatawa12 anatawa12 merged commit c1914f5 into master Oct 12, 2024
6 checks passed
@anatawa12 anatawa12 deleted the rename-blendshape branch October 12, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename BlendShape
2 participants