TokisanGames / Terrain3D

A high performance, editable terrain system for Godot 4.
MIT License
2.21k stars 131 forks source link

Warn if mipmaps are not enabled on texture #542

Open Giwayume opened 2 days ago

Giwayume commented 2 days ago

Terrain3D version

v0.9.3

System information

Godot 4.3 - Steam Deck

Is the issue reproducable in the demo?

Not applicable

Issue description

I was testing on steam deck and ran into a performance issue that basically boiled down to the fact that the texture didn't have mipmaps enabled. It was really bad, from like 60 fps down to 20 fps.

There might be an export type to tell the editor the texture is used for 3d, but if not, would be nice to have a configuration warning somewhere, because adding textures to the terrain doesn't automatically "prepare" them for 3D use like Godot does when you add them to a material.

Logs

No response

TokisanGames commented 2 days ago

You can't mix and match texture configuration, so all textures would have to have mipmaps disabled, which means the user didn't go through the texture preparation documentation. It should be obvious when mipmaps are disabled as the whole screen is a pixelated mess.

{2F536890-34C1-4EF7-9662-3BF40F15FDEA}

Giwayume commented 1 day ago

If the texture is only used for Terrain3D, Godot doesn't auto-enable mipmaps. Also, not obvious on a 4K monitor when the texture that Godot didn't import with mipmaps enabled is a normal map.