Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Direct component access when iterating, benchmarks #93

Merged
merged 20 commits into from
Sep 17, 2023
Merged

Direct component access when iterating, benchmarks #93

merged 20 commits into from
Sep 17, 2023

Conversation

0ffz
Copy link
Member

@0ffz 0ffz commented Sep 13, 2023

  • Changes many areas to read component data directly from memory instead of repacking into an array. This greatly increases performance and chances of getting cache hits.
  • Simplify listener handler definition
  • Other optimizations like using fastutil on jvm, reducing contains checks in archetypes

0ffz added 20 commits August 29, 2023 14:34
@0ffz
Start fiddling with kotlinx.benchmark
e6dfeef
@0ffz
Work on letting accessors get component data directly instead of pack…
e11e0bc 
…ing to array
@0ffz
Add some more unpacking benchmarks, use fast int to int map for compo…
49c1549 
…nent mapping
@0ffz
Cache component accessor index per archetype, don't repack archetype …
0de9597 
…data when moving entity
@0ffz
Avoid componentId calls in common methods, don't bother firing set ev…
f57a7b0 
…ent if no listeners are applicable
@0ffz
Allow component accessor to set directly. Add a velocity system test
6e1be1c
@0ffz
Use EntityTypes's indexOf instead of a map in archetype
bf45f6f
@0ffz
Fix some bugs with event listeners, add new simpler tests for them
7becb5d
@0ffz
Work further on accessor syntax for listeners
2605936
@0ffz
Improve accessor syntax for default, map, removable
768c143
@0ffz
Remove commented out code in some accessors
e12e1eb
@0ffz
Add jvm specific implementation for archetype map, avoid some slow co…
e72a181 
…ntains checks in achetype set component, general cleanup
@0ffz
Iterate to calculate indexOf for EntityType
484f380
@0ffz
Slight cleanup, allow using component accessor without delegate
506ec30
@0ffz
Update addons, rename AccessorThisRef to Pointer
28738fa
@0ffz
Use records typealias, remove ops/s comments
1cd4a99
@0ffz
Update copyright line in license to be more accurate
aa4eb48
@0ffz
A bit of cleanup
56d8c58
@0ffz
Reimplement checking handler, rename fastForEach, add toList, other c…
4e2e8ef 
…leanup
@0ffz
Final bits of cleanup
7ef2702
@0ffz 0ffz merged commit f2e9ea8 into develop Sep 17, 2023
1 check passed
@0ffz 0ffz deleted the direct-component-access branch September 17, 2023 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@0ffz