Memory leak related to Renderer.render() method about imgui-wgpu-rs HOT 9 CLOSED

Just saw this. I've been busy with classes the past two semesters and haven't really had time to follow up on this. Whenever I get the chance I'll try porting my project to 0.7 and see if it helps.

from imgui-wgpu-rs.

Upon doing some profiling in Visual Studio, it appears the leak is related to upload_index_buffer(), upload_vertex_buffer(), and update_uniform_buffer() (Couldn't figure out how to export the heap snapshot, so here's a screenshot : https://i.imgur.com/DRIsHLc.png). Could this be an issue related to the vulkan backend of wgpu-rs rather than a problem with this crate?

EDIT: Also appears to occur with the DX12 and DX11 backends, with DX11 actually leaking faster than the other two.

from imgui-wgpu-rs.

Interestingly enough it appears to only leak in debug mode. When compiling in release the issue is seemingly gone.

from imgui-wgpu-rs.

Hmm odd, I never noticed ... Is this some new occurence?
Does this happen with Vulkan/Metal too?

I'll have to conduct some experiments, but maybe some1 else already knows :)

from imgui-wgpu-rs.

Yeah it happens with Vulkan (haven't tested metal), but it's not as bad as with DX11 and DX12, which have much worse leaks. Again, it seems to not happen on my friends Arch install, so could be an OS specific issue, or possibly even a driver or hardware specific issue. Definitely strange though.

from imgui-wgpu-rs.

Does this still happen with the latest wgpu 0.6 ports? I know that wgpu had some leak issues in 0.5 when using create_buffer_with_data.

from imgui-wgpu-rs.

I think I might be experiencing the same issue under DX12 with the latest wgpu master, I am seeing the memory usage of my program grow very quickly, around 100MB/s, even when idling. According to task manager, GPU memory is growing as well. I will see if the same "only leaks on debug" behavior is present in my code as well, and will report back.

from imgui-wgpu-rs.

I can confirm that using latest wgpu-rs fixes the memory leak. The first commit that fixed the issue is gfx-rs/wgpu-rs@66b900b

from imgui-wgpu-rs.

Awesome! Closing as out of scope. Feel free to reopen once another leak is found.

from imgui-wgpu-rs.