Add addObject as reusable method and use it in Collision to enable kwargs

Author: bakpaul

stlib/geometry/__geometry__.py

@@ -49,8 +49,8 @@ def __init__(self, params: GeometryParameters):
             params.data.generateAttribute(self)
         if(params.dynamicTopology):
             if(params.elementType is not None):
-                addDynamicTopology(self, container = dataclasses.asdict(params.data))
+                addDynamicTopology(self, container = dataclasses.asdict(params.data), **params.kwargs)
             else:
                 raise ValueError
         else:
-            addStaticTopology(self, container = dataclasses.asdict(params.data))
+            addStaticTopology(self, container = dataclasses.asdict(params.data),**params.kwargs)

stlib/geometry/cube.py

@@ -1,5 +1,7 @@
 from stlib.geometry import GeometryParameters
 
+#TODO this is outdated an need to be updated with the InternalDataProvider
+
 class CubeParameters(GeometryParameters):
     def __init__(self,  center, edgeLength, pointPerEdge, dynamicTopology = False):
 

stlib/geometry/extract.py

@@ -6,7 +6,7 @@
 
 from Sofa.Core import Node
 
-
+#TODO enable passing kwargs to this to use addObject instead and again get the benefit from splib
 class ExtractInternalDataProvider(InternalDataProvider):
     destElemType : ElementType
     fromElemType : ElementType

stlib/geometry/sphere.py

@@ -1,5 +1,7 @@
 from stlib.geometry import GeometryParameters
 
+#TODO this is outdated an need to be updated with the InternalDataProvider
+
 class SphereParameters(GeometryParameters):
     def __init__(self,  center, radius, pointPerRad, dynamicTopology = False):
 

stlib/prefabs/visual.py

@@ -2,12 +2,14 @@
 from stlib.core.baseParameters import BaseParameters, Callable, Optional, dataclasses, Any
 from stlib.geometry import Geometry, GeometryParameters
 from stlib.geometry.file import FileParameters
+from splib.core.utils import DEFAULT_VALUE, addObject
+
 from Sofa.Core import Object 
 
 @dataclasses.dataclass
 class VisualParameters(BaseParameters):
-    color : Optional[list[float]] = None
-    texture :  Optional[str] = None
+    color : Optional[list[float]] = DEFAULT_VALUE
+    texture :  Optional[str] = DEFAULT_VALUE
 
     geometry : GeometryParameters = dataclasses.field(default_factory = lambda : GeometryParameters())
     addMapping : Optional[Callable] = None
@@ -18,8 +20,7 @@ def __init__(self, params: VisualParameters):
         BasePrefab.__init__(self, params)
 
         geom = self.add(Geometry, params.geometry)
-        # TODO : handle optional color, texture using DEFAULT_VALUE mechanism (as implemented by Paul)
-        self.addObject("OglModel", color=params.color, src=geom.container.linkpath)
+        addObject(self,"OglModel", "MechanicalState", color=params.color, src=geom.container.linkpath, **params.kwargs)
 
         if params.addMapping is not None:
             params.addMapping(self)