update group APIs

Author: Licini

CHANGELOG.md

@@ -9,10 +9,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+* Added `compas.scene.Scene.add_group()` for adding group.
+* Added `compas.scene.Group.add_from_list()` for adding a list of items to a group.
+
 ### Changed
 
 * Fixed error in `circle_to_compas` from Rhino.
 * Fixed Rhino to Rhino brep serialization.
+* Upated `compas.scene.Group.add()` to pass on group kwargs as default for child items.
 
 ### Removed
 

src/compas/scene/group.py

@@ -11,6 +11,13 @@ class Group(SceneObject):
     ----------
     name : str, optional
         The name of the group.
+    **kwargs : dict, optional
+        Additional keyword arguments to pass on to the child sceneobjects as default values.
+
+    Attributes
+    ----------
+    kwargs : dict
+        The keyword arguments to pass on to the child sceneobjects as default values.
 
     Examples
     --------
@@ -36,7 +43,7 @@ def __new__(cls, *args, **kwargs):
 
     def __init__(self, name, **kwargs):
         super(Group, self).__init__(name=name, **kwargs)
-        self.group_kwargs = kwargs
+        self.kwargs = kwargs
 
     @property
     def __data__(self):
@@ -68,7 +75,7 @@ def add(self, item, **kwargs):
         ValueError
             If the scene object does not have an associated scene node.
         """
-        group_kwargs = self.group_kwargs.copy()
+        group_kwargs = self.kwargs.copy()
         group_kwargs.update(kwargs)
         kwargs = group_kwargs
         return super(Group, self).add(item, **kwargs)

src/compas/scene/scene.py

@@ -119,7 +119,7 @@ def add(self, item, parent=None, **kwargs):
                 del kwargs["context"]  # otherwist the SceneObject receives "context" twice, which results in an error
             if isinstance(parent, Group):
                 # Use the kwargs of the parent group as default values for the child sceneobject
-                group_kwargs = parent.group_kwargs.copy()
+                group_kwargs = parent.kwargs.copy()
                 group_kwargs.update(kwargs)
                 kwargs = group_kwargs
             sceneobject = SceneObject(item=item, context=self.context, **kwargs)  # type: ignore