Mirrors/overte
3
0
Fork 0
mirror of https://github.com/overte-org/overte.git synced 2025-04-20 14:03:55 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Better estimate of avatar centre for zone membership

Browse source
This commit is contained in:
Simon Walton 2019-03-25 15:42:30 -07:00
parent d7a1ecdbb3
commit 46f897b693
3 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
assignment-client/src/avatars/AvatarMixerClientData.cpp
View file

@ -23,9 +23,6 @@
#include "AvatarMixerSlave.h"
// Offset from reported position for priority-zone purposes:
const glm::vec3 AvatarMixerClientData::AVATAR_CENTER_OFFSET { 0.0f, 1.0f, 0.0 };
AvatarMixerClientData::AvatarMixerClientData(const QUuid& nodeID, Node::LocalID nodeLocalID) :
NodeData(nodeID, nodeLocalID) {
// in case somebody calls getSessionUUID on the AvatarData instance, make sure it has the right ID
@ -132,7 +129,7 @@ int AvatarMixerClientData::parseData(ReceivedMessage& message, const SlaveShared
incrementNumOutOfOrderSends();
}
_lastReceivedSequenceNumber = sequenceNumber;
glm::vec3 oldPosition = getPosition();
glm::vec3 oldPosition = _avatar->getCentroidPosition();
bool oldHasPriority = _avatar->getHasPriority();
// compute the offset to the data payload
@ -143,10 +140,10 @@ int AvatarMixerClientData::parseData(ReceivedMessage& message, const SlaveShared
// Regardless of what the client says, restore the priority as we know it without triggering any update.
_avatar->setHasPriorityWithoutTimestampReset(oldHasPriority);
auto newPosition = getPosition();
auto newPosition = _avatar->getCentroidPosition();
if (newPosition != oldPosition || _avatar->getNeedsHeroCheck()) {
EntityTree& entityTree = *slaveSharedData.entityTree;
FindPriorityZone findPriorityZone { newPosition + AVATAR_CENTER_OFFSET } ;
FindPriorityZone findPriorityZone { newPosition } ;
entityTree.recurseTreeWithOperation(&FindPriorityZone::operation, &findPriorityZone);
_avatar->setHasPriority(findPriorityZone.isInPriorityZone);
_avatar->setNeedsHeroCheck(false);

1
assignment-client/src/avatars/AvatarMixerClientData.h
View file

@ -223,7 +223,6 @@ private:
PerNodeTraitVersions _perNodeSentTraitVersions;
std::atomic_bool _isIgnoreRadiusEnabled { false };
static const glm::vec3 AVATAR_CENTER_OFFSET;
};
#endif // hifi_AvatarMixerClientData_h

3
assignment-client/src/avatars/MixerAvatar.h
View file

@ -22,6 +22,9 @@ public:
bool getNeedsHeroCheck() const { return _needsHeroCheck; }
void setNeedsHeroCheck(bool needsHeroCheck = true)
{ _needsHeroCheck = needsHeroCheck; }
// Bounding-box World centre:
glm::vec3 getCentroidPosition() const
{ return getWorldPosition() + _globalBoundingBoxOffset; }
private:
bool _needsHeroCheck { false };