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

fix(T&O): VRMのFirstPersonFlagをCategorizationKeyに追加する #1103

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kaikoga
Copy link
Contributor

@kaikoga kaikoga commented Jun 23, 2024

Fix #1071

(この調子で無限に足し続けるってことにはならないと信じたいです)

@kaikoga kaikoga marked this pull request as ready for review June 23, 2024 08:17
}
#endif
// note: unset will fallback to Auto
return VrmFirstPersonFlag.Auto;
Copy link
Owner

Choose a reason for hiding this comment

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

このままだと手動MergeSkinMedMeshのあとのやつがすべてAutoに化けたりしちゃぃませんかね

少し対処法考えさせてください

Copy link
Owner

Choose a reason for hiding this comment

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

ObjectMappingの方でFirstPerson性を持たせせてあげる必要がありそうかなぁ

Copy link
Owner

Choose a reason for hiding this comment

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

component.Renderers = component.Renderers
.Select(r => new VRMFirstPerson.RendererFirstPersonFlags
{
Renderer = mappingSource.GetMappedComponent(r.Renderer).MappedComponent,
FirstPersonFlag = r.FirstPersonFlag
})
.Where(r => r.Renderer)
.GroupBy(r => r.Renderer, r => r.FirstPersonFlag)

みたいなことをObjectMappingBuilderを少しいじって取得できるようにしてもいいかも

@kaikoga
Copy link
Contributor Author

kaikoga commented Jul 14, 2024

すみません、しばらくかかりそうです!

  • たぶんテストを書いて進めることになります

@kaikoga kaikoga marked this pull request as draft July 14, 2024 03:11
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.

[VRM] T&OのAutoMergeSkinnedMeshがVRMのFirstPerson性を考慮しない
2 participants