search

scenevr / server

:heart_eyes: Create multiuser 3d environments using html-like tags and javascript
http://www.scenevr.com/
BSD 3-Clause "New" or "Revised" License
144 stars 19 forks source link

Get rid of UUIDs, use sequential nodeIds instead #47

Open bnolan opened 9 years ago

bnolan commented 9 years ago

Some problems that came up in #46:

To work with the RDP, we can't use UUIDs, we have to use integer IDs for nodes. We can do this by maintaing a map between UUID and nodeId, but thinking about it, it makes sense to use integers as the node id as a general change to scenevr.

It can be done on the server without changing the client, but since UUIDs will no longer be UUIDs, but instead be a nodeId, I think a larger change should be made to get rid of UUIDs and use nodeId everywhere.

bnolan commented 9 years ago

I think the sequential generator for node ids should be on the document object.

bnolan commented 9 years ago

The RDP project isn't happening now, so this is up for discussion. Sequential IDs make sense because they would be smaller on the wire, the downside is that you can't mash two scenes together as easily (although we don't do that at the moment anyway, since everything is parented to a scene object).

More thought required.