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

ExportRoot を animate する Animation を Export するときの挙動を修正 #775

Merged
merged 2 commits into from
Mar 9, 2021

Conversation

ichi-23
Copy link
Contributor

@ichi-23 ichi-23 commented Mar 9, 2021

  • ExportRoot を animate する Animation を Export するときに KeyNotFoundException が発生する

    • 上記の場合 Export 時に UnityExtensions.GetFromPath の path に空文字が渡され、そのまま UnityExtensions.GetChildByName に空文字が渡されることに起因する
  • 「ExportRoot を animate することはできない」は仕様的に正しいが、上記エラーの発生は意図したものではなさそうなので GetFromPath を修正する

    • path を split するときに空文字を除外することで、path に空文字が渡されたときに自身の Transform を返すようにする
  • GetFromPath を修正したことにより、ExportRoot を animate する Animation を正常に Export できる状態になってしまうので gltf エクスポート時に Validation を追加する

@ichi-23 ichi-23 changed the title ExportRoot を animate uru ExportRoot を animate する Animation を Export するときの挙動を修正 Mar 9, 2021
@@ -213,7 +213,7 @@ public static Transform GetFromPath(this Transform self, string path)
{
var current = self;

var split = path.Split('/');
var split = path.Split(new [] {'/'}, StringSplitOptions.RemoveEmptyEntries);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

StringSplitOptions.RemoveEmptyEntries を使うオーバーロードが Split(char[], StringSplitOptions) しかなかった

@ousttrue ousttrue merged commit 611ec90 into vrm-c:master Mar 9, 2021
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.

3 participants