fix: RPC events sync. - Githubissues

asadm / playroom-unity

9 stars 1 forks source link

fix: RPC events sync. #67

Closed momintlh closed 1 month ago

momintlh commented 1 month ago

RPC events name sync between players. #68
Made Players dictionary private #66

Temporary fix, RPC part needs to be refactored, might be better to handle within JS. #40

SaadBazaz commented 1 month ago

I have a hunch this methodology might cause Race conditions upon high load. Needs testing on high performance calls between multiple players, with multiple RPCs.

Instead of saving a state, maybe send some metadata alongside each RPC call? That way, it's guaranteed to reach along with the RPC call.

SaadBazaz commented 1 month ago

Merging in favor of a working solution instead of over engineering. Noted down problems in Refactor Umbrella issue.