-
Notifications
You must be signed in to change notification settings - Fork 532
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
Migrate to .NET 9 #198
Migrate to .NET 9 #198
Conversation
We should only merge this once .NET 9 has been released globally out of RC, and the version of the Meanwhile, we need more people to test and see what has happened, e.g. performance regression/broken game functionality. |
Pretty sure that's why this was opened as a draft. |
I opened this as a draft precisely because .NET 9 is still in RC; I’ll update the |
17bd1c0
to
9293cdd
Compare
There's some low hanging fruit you could probably add to this PR such as the new A quick find replace on |
Those warnings about |
LGTM. Seems to give a 1% FPS increase on my setup. |
I think all runtime synchronized packages ( |
f1814e5
to
6d6807d
Compare
Improves performance in pretty much every game on my machine (i7-13620H, RTX 4070) Super Mario Odyssey: +4.2 FPS |
@marco-carvalho do you have any reference for "Linq.Any/Linq.FirstOrDefault are now faster than List.Exists/List.Find"? I tried to search but didn't find any existing article or benchmark. On another note, just came across an article claiming |
FPS uplift from 86.5 to 93.4 on Densha de Go Stage Select screen with .NET 9 with frame rates unlocked on my system. I haven't found any issues in my library with this patch. |
d729d17
to
df293f5
Compare
Will this changes break Linux compatibility? |
Gives me around 10-15 more FPS in general Areas on Pokemon Violet - Don't see any regression or issues thus far. |
Uh, no? This PR even has linux builds available. What are you talking about? |
58ca000
to
a382c86
Compare
What's the status on this? Is it ready for merge or are you still looking for ways to apply the new features? |
It is ready for merge since I removed the “draft” status. |
No description provided.