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

Non-consistent out of memory exception #573

Closed
Grossley opened this issue Sep 22, 2021 · 8 comments · Fixed by #1220
Closed

Non-consistent out of memory exception #573

Grossley opened this issue Sep 22, 2021 · 8 comments · Fixed by #1220

Comments

@Grossley
Copy link
Collaborator

I ran UndertaleModTool 7 times, performing the same operation once in the command box, and then attempted to save, and it only successfully saved once.

@Archie-osu
Copy link
Contributor

What was the operation, what game?

@Grossley
Copy link
Collaborator Author

Deltarune chapter 2, Data.GMS2_2_2_302 = false;

@Grossley
Copy link
Collaborator Author

This appears to only occur on "AnyCPU" when running in Visual Studio

@Archie-osu
Copy link
Contributor

And apparently only occurs for some people, as I've been unable to reproduce it even after two hours of trying.

@Grossley
Copy link
Collaborator Author

Use on real hardware not in VM

@DillyzThe1
Copy link

I can confirm that this is an issue, as I always get this error when saving.
And on top of that, I can't even edit or view the decompiled Deltarune Chapter 2 code.

(I'm on Windows 10, running right off the hardware.)

@VladiStep
Copy link
Member

VladiStep commented Nov 15, 2022

It's still relevant.

That's because assemblies for scripts can't be unloaded from memory (at least there's no way that I would aware of), see this issue - dotnet/roslyn#41722

The only way to somehow reduce memory consumption is to call GC.Collect() after every command.

@VladiStep
Copy link
Member

Yeah, GC.Collect() seems to work.

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

Successfully merging a pull request may close this issue.

4 participants