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

[VRM10] FirstPerson の修正 #2252

Merged
merged 3 commits into from
Feb 27, 2024

Conversation

ousttrue
Copy link
Contributor

fixed #2209

  • headless モデルのリソース破棄
  • auto で headless を作成する必要が無かった場合は both
  • firstperson sample の ShowMesh の呼び方

#2209 の修正が主でしたが、関連して問題がある部分も修正しました。

@ousttrue ousttrue added this to the v0.120 milestone Feb 26, 2024
@ousttrue ousttrue added LookAt/FirstPerson create headless model for VR bug Something isn't working labels Feb 26, 2024
Comment on lines +89 to +94
var instance = await Vrm10.LoadPathAsync(path, awaitCaller: awaitCaller, showMeshes: false);

// VR用 FirstPerson 設定
await instance.Vrm.FirstPerson.SetupAsync(instance.gameObject, awaitCaller);

instance.GetComponent<RuntimeGltfInstance>().ShowMeshes();
Copy link
Contributor

Choose a reason for hiding this comment

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

FirstPerson の SetupAsync をどう呼び出さないといけないか?の知識は
Vrm10.Load~~Asyncbool generateFirstPersonModel = false を追加して
Vrm10.Load~~Async の実装の中に入れてしまった方がいいかもしれませんね。

@ousttrue ousttrue merged commit f7aaef0 into vrm-c:master Feb 27, 2024
1 check passed
@ousttrue ousttrue deleted the fix/vrm10_firstperson_auto_dispose branch February 27, 2024 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working LookAt/FirstPerson create headless model for VR
Projects
None yet
2 participants