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

UE4 Hangs/Crashes when updating source control, citing abnormal amounts of GPU usage #86

Closed
SteveChaTTG opened this issue Sep 16, 2021 · 8 comments
Assignees
Labels
bug unreal Unreal Editor bug or missing feature

Comments

@SteveChaTTG
Copy link

SteveChaTTG commented Sep 16, 2021

  1. Connect UE4 to source control
  2. Create a new blueprint file in the Content Browser
  3. Delete the new blueprint files

Result:
UE4 will hang on updating source control and cause abnormal amounts of GPU usage. Might crash editor.

OR

UE4 will now crash if launch with source control enabled. Video of bug available upon request.

@kwljunky
Copy link

I am getting the same issue, but using WWise. An event being renamed, or doing a mass move of files from folder to folder.

@ldmfield
Copy link

I am suffering with this issue atm any news on a fix?

@SRombauts SRombauts self-assigned this Mar 2, 2022
@SRombauts SRombauts added the bug label Mar 2, 2022
@SRombauts SRombauts added the unreal Unreal Editor bug or missing feature label Mar 12, 2022
@SRombauts
Copy link
Owner

This should now be worked around on master with 974ff8f
see also #93 and #97

The workaround has been released as part of https://github.com/SRombauts/UE4PlasticPlugin/releases/tag/1.6.1 on the 2nd of August.

I am now working on a proper fix to be integrated into Unreal Engine 5.1

@SRombautsU SRombautsU reopened this Aug 17, 2022
@SRombauts
Copy link
Owner

I was first able to produce this a few months back when I triggered by mistake a checkout of 23k files in a huge One File Per Actor test project on the cloud
2022-07-27

I have now started reproducing the issue in Unreal without any Source Control operation, with a simple infinite loop in the Provider, meaning it's purely a render thread issue (and I can see how other progress bars in the Editor code seem to handle this)
It crashed my GPU after ~2min07s
2022-08-17 (2)

@SRombauts
Copy link
Owner

A similar bug https://issues.unrealengine.com/issue/UE-93603 Directx 12 RHI can run out of video memory on long process (anims import)

has been fixed by adding End & BeginFrame on the RHI thread https://github.com/EpicGames/UnrealEngine/commit/4cbce19c2848723ccc01a4fba3133d663d64d4ae

@SRombauts
Copy link
Owner

And the official issue would be https://issues.unrealengine.com/issue/UE-100291 "Out of video memory" crash when attempting to save many assets with source control enabled.

A very recent one even mention the workaround I implemented in the plastic plugin! https://issues.unrealengine.com/issue/UE-160299 GPU memory leak during large source control operations:

The crash has been seen in Git and SVN, and can be avoided by removing FScopedSourceControlProgress::Tick. This will prevent Slate from being ticked during the operation, but will prevent the throbber widget from animating.

@SRombauts
Copy link
Owner

SRombauts commented Aug 17, 2022

I opened a PR for Epic Games to integrate the proper fix into Unreal Engine 5.1 hopefully:
https://github.com/EpicGames/UnrealEngine/pull/9492 [UE-100291][UE-160299] Fix source control progress bar crashing out of video memory in case of long-running operation #9492

Cheers!
Sébastien

@SRombauts
Copy link
Owner

The proper fix has been submitted to ue5-main https://github.com/EpicGames/UnrealEngine/commit/67dacf2baaafe0d9b6fb5eeb5b179fb5b871a4bf

It will be part of UE 5.1 release :)

SRombautsU pushed a commit that referenced this issue Jul 13, 2023
…a-space

Fix visual diff failing to getfile if there is a space in the path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unreal Unreal Editor bug or missing feature
Projects
None yet
Development

No branches or pull requests

5 participants