Skip to content

Conversation

@Mugen87
Copy link
Collaborator

@Mugen87 Mugen87 commented May 8, 2025

Reverts #31057

@tatsuya-ogawa I had to revert the PR because the new assets do not just hold a mesh but also a perspective camera and point light. I suspect you have created these in Blender and simply exported everything, not just the mesh. Since the point light will end up as a shadow casting light, this affects the renderer's memory.

Please file a new PR with clean assets.

@Mugen87 Mugen87 closed this May 8, 2025
@Mugen87 Mugen87 reopened this May 8, 2025
@Mugen87 Mugen87 merged commit 8c66673 into dev May 8, 2025
18 of 19 checks passed
@Mugen87 Mugen87 deleted the revert-31057-add_fbx_model_sample branch May 8, 2025 21:23
@tatsuya-ogawa
Copy link
Contributor

tatsuya-ogawa commented May 15, 2025

Sorry to late reply.
Ok I will try later.

tatsuya-ogawa added a commit to tatsuya-ogawa/three.js that referenced this pull request May 15, 2025
Mugen87 pushed a commit that referenced this pull request May 15, 2025
* Revert "Revert "Examples: Add textured assets to `webgl_loader_fbx`. (#31057)" (#31067)"

This reverts commit 8c66673.

* update model to clean mesh
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
…ob#31117)

* Revert "Revert "Examples: Add textured assets to `webgl_loader_fbx`. (mrdoob#31057)" (mrdoob#31067)"

This reverts commit 8c66673.

* update model to clean mesh
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
RuthySheffi pushed a commit to RuthySheffi/three.js that referenced this pull request Jun 5, 2025
…ob#31117)

* Revert "Revert "Examples: Add textured assets to `webgl_loader_fbx`. (mrdoob#31057)" (mrdoob#31067)"

This reverts commit 8c66673.

* update model to clean mesh
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