-
-
Notifications
You must be signed in to change notification settings - Fork 428
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
Band-aid for Bulk SDKHooks performance impact #2 #2094
Conversation
Platform specific changes.
it's the same branch... reopening the old one. ugh. comments in the other PR. Mostly around just tighter levelend code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine, @Headline can you test then merge.
Any progress on testing and mergin this? |
This patch is making a significant difference after multiple tests, thank you for it. |
@Headline Any potential ETA for testing or any way I could help with testing to speed process-up? I personally along with other people in comments so far tested change and have experienced no issues and only performance improvements!
|
Based on the map in #1935 , ported for CS:S Based on the same machine, configs, hardware, os, metamod version (latest). Stock SM:
This commit patch:MM:S 1.12.0-dev+1192 (speed up by 42.7%.)
MM:S 1.11.0-dev+1155 (speed up by 46.52%.)
Stock: profiler__2024-02-06_18-01-09.log |
I uptaded my preivous message with the difference related to MetaMod:Source with and without |
Resolves previously mentioned issues in original pull #2092
Apologies for closing previous one, was going on abruptly planned holidays but decided to grab laptop anyhow so if there are any further issues i'll be able to work further on PR 😅