hertog/godot-module-template
Template
2
1
Fork 1
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "Disable VRAM compression by default for small textures in Detect 3D"

Browse source
This commit is contained in:
Juan Linietsky 2022-06-20 15:30:19 +02:00 • committed by GitHub
parent b4804a2d3f
commit 15837ec191
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 19 additions and 47 deletions
Download patch file Download diff file Expand all files Collapse all files

53
editor/import/resource_importer_texture.cpp
View file

@ -65,15 +65,6 @@ void ResourceImporterTexture::_texture_reimport_3d(const Ref<CompressedTexture2D
}
singleton->make_flags[path].flags |= MAKE_3D_FLAG;
// For small textures, don't use VRAM compression as it decreases quality too much compared to the memory saved.
// The minimum size for VRAM compression is defined on each axis.
// It is then squared to handle non-square input texture sizes in a more human-readable manner.
const float minimum_size = float(GLOBAL_GET("rendering/textures/vram_compression/minimum_size"));
if (p_tex->get_width() * p_tex->get_height() >= int(Math::pow(minimum_size, 2.0f) - CMP_EPSILON)) {
// Texture is larger than `minimum_size × minimum_size` pixels (if square).
singleton->make_flags[path].flags |= MAKE_VRAM_COMPRESS_FLAG;
}
}
void ResourceImporterTexture::_texture_reimport_normal(const Ref<CompressedTexture2D> &p_tex) {
@ -112,33 +103,7 @@ void ResourceImporterTexture::update_imports() {
bool changed = false;
if (E.value.flags & MAKE_3D_FLAG && bool(cf->get_value("params", "detect_3d/compress_to"))) {
if (E.value.flags & MAKE_VRAM_COMPRESS_FLAG) {
// Texture is large enough to benefit from VRAM compression.
const int compress_to = cf->get_value("params", "detect_3d/compress_to");
String compress_string;
if (compress_to == 1) {
cf->set_value("params", "compress/mode", COMPRESS_VRAM_COMPRESSED);
compress_string = "VRAM Compressed (S3TC/ETC/BPTC)";
} else if (compress_to == 2) {
cf->set_value("params", "compress/mode", COMPRESS_BASIS_UNIVERSAL);
compress_string = "Basis Universal";
}
print_line(vformat(TTR("%s: Texture detected as used in 3D. Enabling mipmap generation and setting the texture compression mode to %s."), String(E.key), compress_string));
} else {
print_line(vformat(TTR("%s: Small texture detected as used in 3D. Enabling mipmap generation but not VRAM compression."), String(E.key)));
}
cf->set_value("params", "mipmaps/generate", true);
cf->set_value("params", "detect_3d/compress_to", 0);
changed = true;
}
if (E.value.flags & MAKE_NORMAL_FLAG && int(cf->get_value("params", "compress/normal_map")) == 0 && int(cf->get_value("params", "compress/mode")) != COMPRESS_LOSSLESS) {
// Normal map compression is not available for textures with Lossless compression.
// This is ignored in the importer, but printing a message about normal map compression
// being enabled in this case is misleading.
if (E.value.flags & MAKE_NORMAL_FLAG && int(cf->get_value("params", "compress/normal_map")) == 0) {
print_line(vformat(TTR("%s: Texture detected as used as a normal map in 3D. Enabling red-green texture compression to reduce memory usage (blue channel is discarded)."), String(E.key)));
cf->set_value("params", "compress/normal_map", 1);
changed = true;
@ -151,6 +116,22 @@ void ResourceImporterTexture::update_imports() {
changed = true;
}
if (E.value.flags & MAKE_3D_FLAG && bool(cf->get_value("params", "detect_3d/compress_to"))) {
const int compress_to = cf->get_value("params", "detect_3d/compress_to");
String compress_string;
cf->set_value("params", "detect_3d/compress_to", 0);
if (compress_to == 1) {
cf->set_value("params", "compress/mode", COMPRESS_VRAM_COMPRESSED);
compress_string = "VRAM Compressed (S3TC/ETC/BPTC)";
} else if (compress_to == 2) {
cf->set_value("params", "compress/mode", COMPRESS_BASIS_UNIVERSAL);
compress_string = "Basis Universal";
}
print_line(vformat(TTR("%s: Texture detected as used in 3D. Enabling mipmap generation and setting the texture compression mode to %s."), String(E.key), compress_string));
cf->set_value("params", "mipmaps/generate", true);
changed = true;
}
if (changed) {
cf->save(src_path);
to_reimport.push_back(E.key);