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.

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

Current Arch implementation of EnemyTargetSystem is incomplete and potentially unfair #4

Open
seannowotny opened this issue Jun 1, 2023 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@seannowotny
Copy link
Owner

seannowotny commented Jun 1, 2023

The current implementation makes Arch performance isn't very representative.
I'm waiting for feedback on how I should write this code. PR from anyone who feels like doing this would of course also be great!

@seannowotny seannowotny changed the title Current Arch implementation of EnemyTargetSystem is incomplete Current Arch implementation of EnemyTargetSystem is incomplete and thus misleading Jun 1, 2023
@seannowotny seannowotny changed the title Current Arch implementation of EnemyTargetSystem is incomplete and thus misleading Current Arch implementation of EnemyTargetSystem is incomplete and unfair Jun 1, 2023
@seannowotny seannowotny added the enhancement New feature or request label Jun 2, 2023
@seannowotny seannowotny changed the title Current Arch implementation of EnemyTargetSystem is incomplete and unfair Current Arch implementation of EnemyTargetSystem is incomplete and potentially unfair Jun 2, 2023
@seannowotny
Copy link
Owner Author

Update: There isn't a Unity ECS SharedComponentData or DefaultECS AsMultiMap equivalent in Arch, so a seperate datastructure will be needed to seperate entities with different teams from each other.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant