-
Notifications
You must be signed in to change notification settings - Fork 11
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
Better query syntax, observers, performance improvements #101
Conversation
This file contains 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
…mplementing new query syntax for tracking
- expand upon accessorholder for events
… add deferred syntax instead
- Add syntax for declaring repeating systems
… default name extracted from the creator function Fix: Autoscanning methods not working
… of queriedEntity.row
chore: Clear out some unused classes, suppress some IDE warnings where needed
chore: Clean out some more empty classes or small missed things since main branch
…yOf (#102) perf: Use array implementation of CompId2ArchetypeMap, avoid null boxing using getOrElse perf: Fully remove Record references (unnecessary memory allocs) perf: Use BucketedULongArray to avoid array.copyOf when in ArrayTypeMap, useful when creating many entities at once
… modifications when calling multiple listeners
fix: Error when instantiating prefab that gets modified as it's being created test: Add test for a component set listener running while an entity instantiates test: Add tests for component removal event
…sing function signatures without default values back for now
chore: Use inner serializer for InheritPrefabs fix: not resetting delegated status in forEach call, resulting in incorrect entities returned in queries.
…tPrefabs component instead. We also now use a query to determine what prefabs need to be inherited, so these instances will be included too!
chore: Move PolymorphicListAsMapSerializer vars into a Config glass refactor: Move YamlFileProperties out of YamlFormat, abstract away some logic to let it work with any format
feat: Add deepInstanceOf, collectPrefabs functions on Entity for getting info about the full entity inheritance chain
chore: Move namespaces reading to PolymorphicListAsMapSerializer.Config, make some helper functions for providing the config
tests: Add benchmarks for getting KType and KClass from a reified type to compare componentId overhead
* chore: Cleanup build.gradle js target, add relevant todo * feat: Revamp event listener system to use flecs-style observer pattern * refactor: Tidy up syntax for observers refactor: Reorganize and clean up observer tests * refactor: Remove support for delegating component accessors when using unsafeEntity * perf: Run forEach directly in ArchetypeEventRunner * perf: Memory allocation improvements for observers refactor: Remove no longer needed code from family * refactor: Remove commented out code and some unnecessary class inheritance with accessors * refactor: Fix some IDE inspection issues, removing more unused code * refactor: Replace some manually created datatypes with androidx-collection
feat: Add cacheGroupedBy query fix: Don't double count entities in cacheGrouped/AssociatedBy by observing correct events, and limiting to involved components fix: OnRemove not firing when entity removed
…serializers addon
feat(prefabs): Defining entity observers via config
…, not inheriting observers to instances when created before tracking observer tests(observers): Add entity scoped observer tests
tests(prefabs): Add test for EntityObservers component in config
refactor: Rename CachedQueryRunner -> CachedQuery
0ffz
changed the title
Better query syntax, performance improvements
Better query syntax, observers, performance improvements
May 30, 2024
chore: Add shorthand queries up to size 5 chore: Add family filter to any size query feat: Nullable types in shorthand queries
…t queries perf: Avoid going through delegate entirely in shorthand queries
feat: Add some helper init phase blocks in GearyConfiguration
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
TODO
this
orit
parameter for query operations, some currently use both for backwards compat but this is quite odd.