Robin Rolf

I've just had a user report about triggering this exact assertion failure under a DOS condition (connection spam & "Too many open files" in accept) I *believe* I'm handling this...

> This issue has been open for a while with no activity, has it been resolved? Still would like some clarification if possible

Well done @gopalak for figuring this one out 👍

Oh, usecase The usecase for this would be nicer yielding in api functions while we wait for things to happen/be ready A dialogue system is probably an easy example ```lua...

Yeah, in a final version you'd want to get rid of the lua code parsing and clean up the code all together This is just a proof of concept, it...

NB SyncVar Serialization uses writer.GetWriteFunc which looks up the right method by type: https://github.com/MirrorNetworking/Mirror/blob/master/Assets/Mirror/Editor/Weaver/Processors/NetworkBehaviourProcessor.cs#L446 or defaults to the standard Write method for NetworkBehaviour, which should be NetworkWriterExtensions.WriteNetworkBehaviour if I'm seeing...

Seems to be fixed now in 2022.3.2 preview Creating methods works for me now and it didn't in .1

Oh, the interest management test setup does not play well with this (from memory): - aoi.OnSpawned is not called since test identities are spawned before the aoi is assigned/created -...

Ideally we'd just do a 1:1 non-breaking replacement for the normal spatial hashing here so users don't have to care, that should be doable?

Looking pretty good to me, any chance you can add tests? As a start replacing https://github.com/MirrorNetworking/Mirror/blob/54110660acb630464fd0e3aba76ff4241f078e83/Assets/Mirror/Tests/Editor/SyncCollections/SyncDictionaryTest.cs#L7-L9 with ```csharp [TestFixture(false)] [TestFixture(true)] public class SyncDictionaryTest { private bool optimized; public SyncDictionaryTest(bool optimized)...