Skip to content

Add merge_all_mesh_3d to world_asset_helpers#23710

Open
IceSentry wants to merge 2 commits intobevyengine:mainfrom
IceSentry:merge_world_asset_mesh3d
Open

Add merge_all_mesh_3d to world_asset_helpers#23710
IceSentry wants to merge 2 commits intobevyengine:mainfrom
IceSentry:merge_world_asset_mesh3d

Conversation

@IceSentry
Copy link
Copy Markdown
Contributor

Objective

  • bevy_city needed a way to merge all the Mesh3d from a WorldAsset. Instead of keeping that function hidden inside bevy_city we can make it public

Solution

  • Add world_asset_helpers to bevy_dev_tools. It's in bevy_dev_tools because it depends on multiple different parts of bevy and that's the only crate that users can use that already depends on a lot of things.
  • Add marge_all_mesh_3d() to world_asset_helpers
  • Update bevy_city to use that helper

Testing

  • I ran bevy_city and the meshes were merged like they should

@IceSentry IceSentry added A-Dev-Tools Tools used to debug Bevy applications. C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward X-Contentious There are nontrivial implications that should be thought through labels Apr 7, 2026
@IceSentry
Copy link
Copy Markdown
Contributor Author

I added the contentious label because I expect some people will not like to put this in bevy_dev_tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Dev-Tools Tools used to debug Bevy applications. C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward X-Contentious There are nontrivial implications that should be thought through

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants