Merge pull request #48 from specklesystems/izzy/receive-fixes

fix(cols): try catch linking children

README.md

@@ -1,5 +1,5 @@
 # SpeckleBlender 2.0
-Speckle add-on for Blender 2.92
+Speckle add-on for Blender 2.92 & 2.93
 
 [![Twitter Follow](https://img.shields.io/twitter/follow/SpeckleSystems?style=social)](https://twitter.com/SpeckleSystems) [![Community forum users](https://img.shields.io/discourse/users?server=https%3A%2F%2Fdiscourse.speckle.works&style=flat-square&logo=discourse&logoColor=white)](https://discourse.speckle.works) [![website](https://img.shields.io/badge/https://-speckle.systems-royalblue?style=flat-square)](https://speckle.systems) [![docs](https://img.shields.io/badge/docs-speckle.guide-orange?style=flat-square&logo=read-the-docs&logoColor=white)](https://speckle.guide/dev/)
 
@@ -45,7 +45,7 @@ This code is WIP and as such should be used with extreme caution on non-sensitiv
 ## Custom properties
 
 - **SpeckleBlender** will look for a `texture_coordinates` property and use that to create a UV layer for the imported object. These texture coordinates are a space-separated list of floats (`[u v u v u v etc...]`) that is encoded as a base64 blob. This is subject to change as **SpeckleBlender** develops.
-- If a `material` property is found, **SpeckleBlender** will create a material named using the sub-property `material.name`. If a material with that name already exists in Blender, **SpeckleBlender** will just assign that existing material to the object. This allows geometry to be updated without having to re-assign and re-create materials.
+- If a `renderMaterial` property is found, **SpeckleBlender** will create a material named using the sub-property `renderMaterial.name`. If a material with that name already exists in Blender, **SpeckleBlender** will just assign that existing material to the object. This allows geometry to be updated without having to re-assign and re-create materials.
 - Vertex colors are supported. The `colors` list from Speckle meshes is translated to a vertex color layer.
 - Speckle properties will be imported as custom properties on Blender objects. Nested dictionaries are expanded to individual properties by flattening their key hierarchy. I.e. `propA:{'propB': {'propC':10, 'propD':'foobar'}}` is flattened to `propA.propB.propC = 10` and `propA.propB.propD = "foobar"`.
 

bpy_speckle/convert/__init__.py

@@ -9,6 +9,7 @@ from bpy_speckle.functions import _report, get_scale_length
 from specklepy.objects.geometry import *
 from specklepy.objects.other import RenderMaterial
 
+
 FROM_SPECKLE_SCHEMAS = {
     Mesh: import_mesh,
     Brep: import_brep,
@@ -48,13 +49,14 @@ def add_blender_material(smesh, blender_object) -> None:
     if blender_object.data is None:
         return
 
-    if not hasattr(smesh, "renderMaterial"):
+    if not hasattr(smesh, "renderMaterial") and not hasattr(smesh, "@renderMaterial"):
         return
 
-    speckle_mat = smesh.renderMaterial
-    mat_name = getattr(speckle_mat, "name", None)
+    speckle_mat = getattr(smesh, "renderMaterial", None) or smesh["@renderMaterial"]
+    mat_name = getattr(speckle_mat, "name", None) or speckle_mat.__dict__.get("@name")
     if not mat_name:
         mat_name = speckle_mat.applicationId or speckle_mat.id
+
     blender_mat = bpy.data.materials.get(mat_name)
     if not blender_mat:
         blender_mat = bpy.data.materials.new(mat_name)
@@ -196,6 +198,7 @@ def from_speckle_object(speckle_object, scale, name=None):
         if speckle_name in bpy.data.objects.keys():
             blender_object = bpy.data.objects[speckle_name]
             blender_object.data = obdata
+            blender_object.matrix_world = Matrix()
             if hasattr(obdata, "materials"):
                 blender_object.data.materials.clear()
         else:

bpy_speckle/convert/from_speckle/mesh.py

@@ -157,7 +157,7 @@ def import_mesh(speckle_mesh, scale=1.0, name=None):
     if not name:
         name = speckle_mesh.geometryHash or speckle_mesh.id
 
-    if name in bpy.data.meshes.keys() and False:
+    if name in bpy.data.meshes.keys():
         mesh = bpy.data.meshes[name]
     else:
         mesh = bpy.data.meshes.new(name=name)

bpy_speckle/operators/streams.py

@@ -286,8 +286,10 @@ class ReceiveStreamObjects(bpy.types.Operator):
             bpy.context.scene.collection.children.link(col)
 
         for child_col in collections.keys():
-            col.children.link(bpy.data.collections[child_col])
-
+            try:
+                col.children.link(bpy.data.collections[child_col])
+            except:
+                pass
         """
         Set conversion scale from stream units
         """

bpy_speckle/operators/users.py

@@ -55,6 +55,8 @@ class LoadUsers(bpy.types.Operator):
             except Exception as ex:
                 _report(ex)
                 users.remove(len(users) - 1)
+            if profile.isDefault:
+                context.scene.speckle.active_user = str(len(users) - 1)
 
         context.scene.speckle.active_user_index = int(context.scene.speckle.active_user)
         bpy.ops.speckle.load_user_streams()