-
Notifications
You must be signed in to change notification settings - Fork 75
Conversation
a91451c
to
6028cfa
Compare
b32000a
to
36e881f
Compare
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
cfad802
to
46fb919
Compare
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing the ECS upgrade as well :D
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
var clientRangeInterest = InterestQuery.Query(Constraint.RelativeCylinder(clientRadius)).FilterResults(new[] | ||
{ | ||
Position.ComponentId, Metadata.ComponentId, OwningWorker.ComponentId, | ||
ServerMovement.ComponentId, ClientRotation.ComponentId, HealthComponent.ComponentId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do players need other player's health?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup, RagdollSpawner
requires it
var serverSelfInterest = InterestQuery.Query(Constraint.EntityId(entityId)).FilterResults(new[] | ||
{ | ||
ClientMovement.ComponentId, ShootingComponent.ComponentId | ||
}); | ||
|
||
var serverRangeInterest = InterestQuery.Query(Constraint.RelativeCylinder(serverRadius)).FilterResults(new[] | ||
{ | ||
Position.ComponentId, Metadata.ComponentId, OwningWorker.ComponentId, | ||
ServerMovement.ComponentId, ClientRotation.ComponentId, HealthComponent.ComponentId, | ||
ShootingComponent.ComponentId | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm having a little trouble understanding why we chose these sets for the server interest, a comment could help? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, will add
Signed-off-by: Paul Balaji <paulbalaji@improbable.io>
Description
Add component filters to qbi queries
drive-by upgrade to latest entities package too
Tests
Documentation
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.