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

[export][springbone] bake 時に SpringBone と LookAt に反映する実装 #2468

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

ousttrue
Copy link
Contributor

@ousttrue ousttrue commented Oct 18, 2024

freeze_pose 時に SpringBone と LookAtOffset を反映する機能を実装しました。

fixed #2426
fixed #2277
fixed #2368

@ousttrue ousttrue added this to the next export bake 関連 milestone Oct 18, 2024
@ousttrue ousttrue added exporter SpringBone LookAt/FirstPerson create headless model for VR mod_export For example, remove bone Non breaking changes labels Oct 18, 2024
@ousttrue ousttrue requested a review from Santarh October 21, 2024 07:57
Copy link
Contributor

@notargs notargs left a comment

Choose a reason for hiding this comment

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

コンフリクト修正おねがいします 🙇

@s-iwaki-d s-iwaki-d removed the request for review from Santarh October 21, 2024 09:03
@ousttrue
Copy link
Contributor Author

rebase しました 🙏

@ousttrue ousttrue requested a review from notargs October 21, 2024 13:47
@@ -208,8 +208,11 @@ private static byte[] Export(

if (settings.PoseFreeze)
{
// 正規化
VRMBoneNormalizer.Execute(target, settings.ForceTPose, settings.FreezeMeshUseCurrentBlendShapeWeight);
using (var backup = new VrmGeometryBackup(target))
Copy link
Contributor Author

@ousttrue ousttrue Oct 21, 2024

Choose a reason for hiding this comment

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

using の最初に SpringBone 関連の world 位置を記録。
Dispose で復帰(正規化で変わっている可能性がある)。

// SkinnedMeshRenderer.sharedMesh と MeshFilter.sharedMesh を新しいMeshで置き換える
BoneNormalizer.Replace(root, newMeshMap, m_settings.FreezeMeshKeepRotation);
}
using (var backup = new Vrm10GeometryBackup(root))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

using の最初に SpringBone 関連の world 位置を記録。
Dispose で復帰(正規化で変わっている可能性がある)

Copy link
Contributor

@notargs notargs left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ousttrue ousttrue merged commit 87eef5e into vrm-c:master Oct 22, 2024
1 check passed
@ousttrue ousttrue deleted the fix/impl_springbone_bake branch October 22, 2024 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants