iTwin / itwinjs-core

Monorepo for iTwin.js Library
https://www.itwinjs.org
MIT License
591 stars 211 forks source link

iModel transformations do not remap/preserve texture references. #4947

Closed MichaelBelousov closed 1 year ago

MichaelBelousov commented 1 year ago

Describe the bug iModel transformations do not remap/preserve texture references.

To Reproduce Create an iModel with some geometry that contains a texture, and use the imodel-transformer test app to identity-transform it into a new file, textures will be gone if you open it in a viewer. This applies to normal maps, patterns, diffuse textures (see below for a more specific repro).

Expected behavior Textures are preserved in an identity transformation.

Desktop (please complete the applicable information):

pmconne commented 1 year ago

More specifically (correct me if you disagree @MichaelBelousov):

RenderingAsset::Relocate looks completely broken.