Skip to content

Performance Improvements - #56

Merged
aonkeeper4 merged 3 commits into
CommunalHelper:devfrom
karl2883:dev
Sep 1, 2026
Merged

Performance Improvements#56
aonkeeper4 merged 3 commits into
CommunalHelper:devfrom
karl2883:dev

Conversation

@karl2883

Copy link
Copy Markdown

Two performance improvements:

  • add a ContainsEntity function that checks if an entity container contains an entity without allocations. Using GetEntities().Contains() as before allocates a new List each time, which is costly in hot paths
  • Rewrite the Collide.Check IL hook to only check for EeveeHelper relationships if the entities would actually collide - in an ordinary level, the amount of collision checks that fail because there is no collision should be vastly greater than the ones that fail because there is an EeveeHelper relationship, so this is cheaper.

@aonkeeper4 aonkeeper4 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, will run the SJ tas to be sure.

@aonkeeper4

Copy link
Copy Markdown

Seems to be all good.

@aonkeeper4
aonkeeper4 merged commit dcc25da into CommunalHelper:dev Sep 1, 2026
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 this pull request may close these issues.

2 participants