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

Synchronization of group properties #10

Open
kittyfly opened this issue Dec 27, 2012 · 1 comment
Open

Synchronization of group properties #10

kittyfly opened this issue Dec 27, 2012 · 1 comment

Comments

@kittyfly
Copy link
Collaborator

This is a non-urgent issue that might be a problem later, so documenting it here.

Object group properties (the properties that are only meaningful in SOG, not in SOP), such as group position, better be only sync'ed for root prim in the group. That is, similar to physical properties that only exist in the RegionModule's sync cache for physical prims, group properties should only exist for root prims in the sync cache. For non-root prims, CurrentlySyncedProperties.ContainsKey(property) == false, if the property is a group property.

When synchronizing a group property, the sender only sends out a sync message for the root prim. Similarly, the receiver should only update the group property in the sync cache of the root prim.

@dslake
Copy link
Collaborator

dslake commented Dec 27, 2012

In the current version, I think group properties are only sent for root prims but they are definitely stored in the cache for all prims which is unnecessary. I'll confirm this and track the issue of storing group properties for non-root prims. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants