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

Attempting to edit ShaderMaterial leads to editor freezing #49937

Closed
Chaosus opened this issue Jun 27, 2021 · 6 comments · Fixed by #49973
Closed

Attempting to edit ShaderMaterial leads to editor freezing #49937

Chaosus opened this issue Jun 27, 2021 · 6 comments · Fixed by #49973

Comments

@Chaosus
Copy link
Member

Chaosus commented Jun 27, 2021

Godot version

92f7504 bde6f5e

System information

Windows 10

Issue description

Attempting to edit a shader cause editor freeze with the following spam to console:

image

The error is:

ERROR: Condition "err" is true. Returning: ERR_CANT_CREATE
   at: VulkanContext::prepare_buffers (drivers\vulkan\vulkan_context.cpp:1728)

call stack:

 	godot.windows.opt.tools.64.exe!VulkanContext::prepare_buffers() Line 1729	C++
 	godot.windows.opt.tools.64.exe!RenderingDeviceVulkan::prepare_screen_for_drawing() Line 8137	C++
 	godot.windows.opt.tools.64.exe!RendererViewport::draw_viewports() Line 623	C++
 	godot.windows.opt.tools.64.exe!RenderingServerDefault::_draw(bool p_swap_buffers, double frame_step) Line 115	
 	godot.windows.opt.tools.64.exe!Main::iteration() Line 2539	C++
 	godot.windows.opt.tools.64.exe!OS_Windows::run() Line 622	C++
 	godot.windows.opt.tools.64.exe!widechar_main(int argc, wchar_t * * argv) Line 163	C++
 	godot.windows.opt.tools.64.exe!_main() Line 187	C++
>	godot.windows.opt.tools.64.exe!main(int argc, char * * argv) Line 199	C++

Steps to reproduce

Create a simple scene with a mesh, assign ShaderMaterial to material override property and try open it.

Minimal reproduction project

ShaderTest.zip

@Chaosus Chaosus added this to the 4.0 milestone Jun 27, 2021
@SaracenOne
Copy link
Member

This seems to happen with any material type, not just ShaderMaterial. It can also happen if you simply create the material from the inspector window without having to assign it a MeshInstance.

@Chaosus
Copy link
Member Author

Chaosus commented Jun 27, 2021

This seems to happen with any material type, not just ShaderMaterial.

Yeah, however, all other types of material are just shells for shader material and compile to it internally.

@Chaosus
Copy link
Member Author

Chaosus commented Jun 28, 2021

I've bisected it. Its a regression from bde6f5e

@Chaosus Chaosus changed the title Attempting to edit ShaderMaterial cause editor stuck Attempting to edit ShaderMaterial leads to editor freezing Jun 28, 2021
reduz added a commit to reduz/godot that referenced this issue Jun 28, 2021
* Multisampling was wrongly selected, possibly fixes godotengine#49937
* Image semaphore acquisition is now per window, possibly fixes godotengine#41614

Please make sure to test the above two issues again, since I can't reproduce either anyway.
@akien-mga
Copy link
Member

Please confirm that the freeze is indeed fixed with #49973 if you can.

@SaracenOne
Copy link
Member

@akien-mga Just tested this and it doesn't appear to be crashing anymore, yeah!

@Chaosus
Copy link
Member Author

Chaosus commented Jun 29, 2021

Indeed, it's fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants