decentraland / unity-explorer

Apache License 2.0
10 stars 11 forks source link

[QA] Emote 2.0 texture quality reduced in new client #1846

Open Ludmilafantaniella opened 3 months ago

Ludmilafantaniella commented 3 months ago

Explorer Alpha build versions:

Issue Description:

The Emote 2.0, which uses a single baked 2048x2048 texture, renders at full quality in the old client but appears significantly reduced in quality in the new client. The texture seems to be rendered at a lower resolution, possibly down to 512x512, despite settings being on low or ultra. The issue is also present in Opera GX.

This also happens on other Emotes 2.0 like:

METATIGER-Quality bug

STR:

  1. Launch the new client.
  2. Equip the Emote 2.0 with a 2048x2048 texture.
  3. Compare the texture quality in the new client with the old client and in Opera GX..

Player LOGS:

(METATIGER) Drop in texture quality for Emotes 2.0 .zip

lorux0 commented 2 months ago

I’ve been checking the documentation and we do require textures 512x512 for scenes and wearables, but not for emote props, although it mentions these textures limitations. We have decided in the team to not modify these limitations since it would increase the memory consumption. The creator will have to fix the emote. The quality is better in the old desktop client since it doesnt use asset bundles, but the original glb instead.

lorux0 commented 2 months ago

After reviewing the issue with the team again, we have decided to change the limit to 1024x1024 only for explorer-alpha. The quality improvement is noticeable. This is the pr: https://github.com/decentraland/asset-bundle-converter/pull/157 Additionally we need to update the official documentation.