Don't mark instance meshes dirty every update. We can use the hash to only send them when needed and only send instance infos every udpate.

Author: schinkowski

Plugins~/Src/MeshSyncClient/msInstancesManager.cpp

@@ -65,27 +65,22 @@ namespace ms {
     void InstancesManager::add(InstanceInfoPtr info)
     {
         auto& rec = lockAndGet(info->path);
-
-        bool existed = false;
-
+        
         for (int i = 0; i < rec.instancesPerParent[info->parent_path].size(); ++i)
         {
             const auto& instance = rec.instancesPerParent[info->parent_path][i];
 
             if (instance->parent_path == info->parent_path)
             {
-                existed = true;
-
                 // Remove instance from the list, the new one will be added below:
                 rec.instancesPerParent[info->parent_path].erase(rec.instancesPerParent[info->parent_path].begin() + i);
 
                 break;
             }
         }
 
-        if (m_always_mark_dirty || !existed) {
-            rec.dirtyInstances = true;
-        }
+        // instanceInfos need to be sent every update because it's how the server knows which instances are still alive:
+        rec.dirtyInstances = true;
 
         rec.instancesPerParent[info->parent_path].push_back(info);
         rec.updatedParents[info->parent_path] = true;

Plugins~/Src/MeshSyncClientBlender/msblenContextGeometryNodes.cpp

@@ -37,10 +37,7 @@ void msblenContext::exportInstances() {
             auto id = static_cast<ID*>(obj->data);
             scene_objects.insert(id->name + 2);
         });
-
-    // Assume everything is now dirty
-    m_instances_state->manager.setAlwaysMarkDirty(true);
-
+    
     std::unordered_map<std::string, ms::TransformPtr> exportedTransforms;
 
     m_geometryNodeUtils.each_instanced_object(