🦟 Bug fix

Summary

https://github.com/gazebosim/gz-common/pull/563 rebased to main.

When importing assimp scenes, the name of the generated texture used the name of the root node as a prefix. This worked well in simple GLB assets where there is only one node and the name would be its name but the moment a user was to import a scene with more than one node assimp would automatically create a new node and hardcode its name to ROOT as shown here.

This creates issues when running gz with ogre2 since we use the texture name to share textures between different assets, so different files would result in textures called ROOT_[...] and would cause the wrong texture to be displayed.

The fix is to use the scene name instead of the root node name (which was probably the right implementation in the first place).

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

gazebosim / gz-common

Use scene name for material name #564

🦟 Bug fix

Summary

Checklist

Codecov Report