Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect 3D at import-time treats normal/AO/height maps as roughness maps, but does not treat roughness maps as such #61498

Open
Calinou opened this issue May 28, 2022 · 1 comment

Comments

@Calinou
Copy link
Member

Calinou commented May 28, 2022

Godot version

4.0.alpha8

System information

Fedora 36, GeForce GTX 1080 (NVIDIA 510.68.02)

Issue description

When configuring textures in a StandardMaterial3D, normal map, ambient occlusion map and height maps are all detected to be used as a roughness map, which is incorrect. The roughness map is not detected as being used as a roughness map.

I haven't checked behavior with ORMMaterial3D, but it's likely affected too.

Steps to reproduce

  • Ensure .godot/ folder is not present within the project folder, and ensure *.import files are not present next to files. This forces 3D detection to happen again, which is important in case you're looking into fixing this.
  • Open the minimal reproduction project in the editor (.godot/ and *.import files were already removed from it).
  • Look at textures' import flags by selecting them in the FileSystem dock, then looking at the Import dock.

Minimal reproduction project

test_material_detect_3d.zip

@lyuma
Copy link
Contributor

lyuma commented Feb 27, 2023

Some of these issues may have been solved already. For me, the Normal and AO maps do not seem to be detected as Roughness. Normal's roughness setting changes to Disabled. The others remain on Detect.

The height (displacement) map is being detected to depend on the normal map, but the roughness value remains on Detect.

Roughness map also remains as detect even if I re-assign it into the roughness slot. This is probably the main issue here, though interestingly it's not assigning the Normal map into the source normal slot of roughness, even though it did so for displacement.

@lyuma lyuma modified the milestones: 4.0, 4.1 Feb 27, 2023
@YuriSizov YuriSizov modified the milestones: 4.1, 4.2 Jun 22, 2023
@YuriSizov YuriSizov modified the milestones: 4.2, 4.x Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants