Various performance related commits carried over from TC2. #468
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
it's only being used by a single effect. this is slower, worse for cache, and less accurate than using the builtins. why software emulate a table? to ensure backwards compat, i kept the functions and just redirected them to the function calls. but i did remove the constants, if anyone used these I don't think they can/should be helped Co-authored-by: mastercoms <mastercoms@tuta.io>
instead of doing a O(n) lookup per frame in some cases, we can just init the ConVarRef once these ones are just possible during runtime or called relatively more compared to other non-static ConVarRefs there is one in R_LoadSkys for skyname as well, but not sure if that was fixed or not Co-authored-by: mastercoms <mastercoms@tuta.io>
Co-authored-by: mastercoms <mastercoms@tuta.io>
Co-authored-by: mastercoms <mastercoms@tuta.io>
* lower LODs are now marked as set up * align parent transform matrix Co-authored-by: mastercoms <mastercoms@tuta.io>
this skips a pre-mature/extra bone setup during particle simulation and flexing by allowing for the last frame's attachment position to be used technically this is a bit of a hack but it works well from my analysis. it gets rid of almost all of the particle cost in team fights besides sprite rendering Co-authored-by: mastercoms <mastercoms@tuta.io>
cheap water LOD was disabled with local specular due to shipping constraints but re-enabled for ep2 on xbox 360 this change now re-enables it for all platforms Co-authored-by: mastercoms <mastercoms@tuta.io>
some players report that they have tracked down some cases where the player model could still update its rendering outside the class menu this is commonly observed through FPS increasing when using the scoreboard may only be relevant for some custom HUDs Co-authored-by: mastercoms <mastercoms@tuta.io>
g_ragdoll_fadespeed was not being used in TF2 ragdolls despite being used in other games fix cl_ragdoll_forcefade not working on the same frame if delay is 0 fix cosmetics not respecting ragdoll fade settings Co-authored-by: mastercoms <mastercoms@tuta.io>
despite being a less efficient renderer, DX8 is still used for performance reasons due to its lower quality settings. we can continue to provide these lower quality settings with the much more robust and efficient DX9 renderer, which will benefit the playerbase and game. engine change: mastercomfig/tf2-patches-old@eaac092#diff-260f8c14c7dd12646fa64721dcdf647450005b5002ce33047bc6a9ccb578daac Co-authored-by: mastercoms <mastercoms@tuta.io>
Co-authored-by: mastercoms <mastercoms@tuta.io>
this fixes platform inconsistencies between Linux and Windows on certain floating point values. for example, on Windows servers, small ammo packs will grant 41 metal when on Linux servers they provide the intended 40 metal. Ceil2Int is also faster computationally Co-authored-by: mastercoms <mastercoms@tuta.io>
Blixibon
approved these changes
Sep 4, 2025
Member
Blixibon
left a comment
Blixibon
left a comment
There was a problem hiding this comment.
I know of a few HL2 modders who have tried and failed to restore water_lod_control on expensive water over the years. I don't know much about it, nor do I necessarily know what it's supposed to look like, but I tested it in-game and it does indeed seem to fade out to cheap water at the specified distances... I guess I'll pass that on 🥴
That and everything else in this PR looks good to me 👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains various performance related commits carried over from Team Comtress 2 with permission.
What each commit does
This list describes each commit in this PR:
General performance:
TableCosfunction inmathlib.hto useFastCos. Using a table is slower, worse for cache, and less accurate than using the builtins.water_lod_control.Ceil2IntandFloor2Int.Team Fortress 2 specific:
tf_impactwatertimeenableandtf_impactwatertimeand provide better descriptions.g_ragdoll_lvfadespeedandg_ragdoll_fadespeedon TF2's ragdolls. Also fixescl_ragdoll_forcefadenot working on the same frame if delay is 0, and cosmetics occasionally not respecting the fade.PR Checklist
developbranch OR targets another branch with a specific goal in mind