Skip to content
This repository has been archived by the owner on Jul 16, 2024. It is now read-only.

Archetype negation #93

Closed
wants to merge 36 commits into from
Closed

Archetype negation #93

wants to merge 36 commits into from

Conversation

Ukendio
Copy link
Contributor

@Ukendio Ukendio commented Dec 23, 2023

Addresses #91 but depends on #90 before this is implementable.

In every case performs better than existing queries. Scales relatively to how many entities you filter from :without
image
image

Benchmark file: https://drive.google.com/file/d/1JJbPvVdSCS0kRd6HgMQ2bGEhDmhk1kmy/view?usp=sharing

Passes every unit test currently, but waiting to hear feedback from others to test it in real production code.

@memorycode
Copy link
Contributor

These changes caused the system with this query to go from 25μs -> 5μs. There are about 75 entities with Components.A and Components.B:

world:query(Components.A, Components.B):without(Components.C)

@Ukendio
Copy link
Contributor Author

Ukendio commented Jan 7, 2024

Moved to matter-ecs/matter#4

@Ukendio Ukendio closed this Jan 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants