-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Test babylon mesh lod 2 #13234
Test babylon mesh lod 2 #13234
Conversation
The example in the documentation (https://doc.babylonjs.com/features/featuresDeepDive/mesh/LOD): > You can remove a LOD level by using removeLODLevel: > > ``` > knot00.removeLODLevel(knot02); > knot00.removeLODLevel(null); > ``` But this signature with `null` was impossible.
Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s). |
Snapshot stored with reference name: Test environment: To test a playground add it to the URL, for example: https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13234/merge/index.html#WGZLGJ#4600 Links to test babylon tools with this snapshot: https://playground.babylonjs.com/?snapshot=refs/pull/13234/merge To test the snapshot in the playground with a playground ID add it after the snapshot query string: https://playground.babylonjs.com/?snapshot=refs/pull/13234/merge#BCU1XR#0 |
Thanks a lot @Dok11 |
Test babylon mesh lod 2 Former-commit-id: 594b70b558abb1abbbe5030619675e62003c98a2
Hi!
Here I found some trouble with
removeLODLevel
method — in the doc presented example where we can removenull
from LOD, but the method didn't have ability to do it.https://doc.babylonjs.com/features/featuresDeepDive/mesh/LOD
Also I did not found the way how to reach this code part:

It's necessary? In which cases?
And one thing more. I dont know is it trouble or not, but the
addLODLevel
method allows to add multiple meshes to the same distance. There are no any checks to this case.Probably have reaseon to add Logger Warn at least?