- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 110
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
Combine all changes #1551
Combine all changes #1551
Conversation
This looks incredible @TimothyMakkison Did you have a benchmark before and after any of your changes so we can estimate the total change? |
Thanks 😄 optimisations are my favourite changes, very satisfying. OOC how did you find this PR?
See #1552 for benchmark performance before I made any changes. The before section of #1550 is the current performance. Note that my changes only apply to the formatting logic, csharpier uses a caching system and only formats files that changed. This will be most beneficial when working on several medium to large files. IIRC the bulk of the time running Original #1552
|
I've been meaning to learn about performance and improving that in .NET. I was curious if you had any resources/courses that you'd recommend? I have been 'watching' this repo on GitHub so I just see the list of PRs in my GH notifications, and I've been seeing all your awesome PRs in awe. Thanks for that info and the old link - looks like pretty awesome speedups. I can't wait :) |
I don't know what your day job is, but most day to day CRUD apps are bottlenecked by: IO, HTTP calls, bad mulithreading and database calls/SQL. These optimisation are more applicable for libraries or high performance scenarios. I'm not an expert but I'd recommend:
|
All my local/pending changes, I have a couple more ideas but this should give a general feel for the final performance