Remove Sofa.Components (#456)

* Remove Sofa.Components.

I suspect this code to have been integrated in the SofaPython3 by mistake and to have never
worked. By looking at it it may be an experiment to allow syntax like this

import Sofa.Components
Sofa.Components.MechanicalObject()  ...

* Example on how to fix pybind11 type forward declaration so stubgens knows about them.

The problem:
Depending on the definition order of the binded classes, there may have
incorrect types if Base is useing BaseData... but BaseData is only binded after Base.

The PR propose a solution for that using a decidcated "forward" registration patter.

* Update tests to take into account removal of Sofa.Components.

* FIXUP

Author: damienmarchal

bindings/Modules/tests/SofaConstraintSolver/matrix_access.py

@@ -1,6 +1,5 @@
 import unittest
 import Sofa.Core
-import Sofa.Components
 from Sofa import SofaConstraintSolver
 
 class Test(unittest.TestCase):

bindings/Modules/tests/SofaLinearSolver/matrix_access.py

@@ -1,6 +1,5 @@
 import unittest
 import Sofa.Core
-import Sofa.Components
 from Sofa import SofaLinearSolver
 
 class Test(unittest.TestCase):

bindings/Sofa/Bindings.SofaConfig.cmake.in

@@ -8,7 +8,7 @@ find_package(SofaPython3 QUIET REQUIRED COMPONENTS Plugin)
 # Required by Bindings.Sofa.Helper, Bindings.Sofa.Types
 find_package(Sofa.Core QUIET REQUIRED)
 
-# Required by Bindings.Sofa.Core, Bindings.Sofa.Components
+# Required by Bindings.Sofa.Core
 find_package(Sofa.Simulation.Core QUIET REQUIRED)
 
 # Required by Bindings.Sofa.Core

bindings/Sofa/CMakeLists.txt

@@ -1,7 +1,6 @@
 project(Bindings.Sofa)
 
 set(SOFABINDINGS_MODULE_LIST
-    Components
     Core
     Helper
     Simulation

bindings/Sofa/package/__init__.py

@@ -161,7 +161,6 @@
 import Sofa.Core
 import Sofa.Simulation
 import Sofa.Types
-import Sofa.Components
 import SofaTypes
 
 from .prefab import *

bindings/Sofa/src/SofaPython3/Sofa/Components/CMakeLists.txt

@@ -34,6 +34,8 @@ using sofa::helper::WriteOnlyAccessor;
 
 #include <SofaPython3/PythonFactory.h>
 
+#include <SofaPython3/Sofa/Core/Binding_BaseData.h>
+#include <SofaPython3/Sofa/Core/Binding_BaseLink.h>
 #include <SofaPython3/Sofa/Core/Binding_LinkPath.h>
 #include <SofaPython3/Sofa/Core/Binding_Base.h>
 #include <SofaPython3/Sofa/Core/Binding_Base_doc.h>
@@ -449,6 +451,9 @@ py::object BindingBase::setDataValues(Base& self, py::kwargs kwargs)
 
 void moduleAddBase(py::module &m)
 {
+    moduleForwardAddBaseData(m);
+    moduleForwardAddBaseLink(m);
+
     py::class_<Base, py_shared_ptr<Base>> base(m, "Base", py::dynamic_attr(), doc::base::BaseClass);
     /// set & get the name as string. The alternative is to access the data field using
     /// obj.name.value = "aName"

bindings/Sofa/src/SofaPython3/Sofa/Components/Submodule_Components.cpp

@@ -192,10 +192,24 @@ std::string getValueTypeString(BaseData* data)
     return data->getValueTypeInfo()->name();
 }
 
+auto getPythonClassForBaseData(py::module& m)
+{
+    /// Register the BaseData binding into the pybind11 system.
+    static py::class_<BaseData, std::unique_ptr<sofa::core::objectmodel::BaseData, pybind11::nodelete>> data(m, "Data", sofapython3::doc::baseData::BaseDataClass);
+    return data;
+}
+
+void moduleForwardAddBaseData(py::module& m)
+{
+    getPythonClassForBaseData(m);
+}
+
 void moduleAddBaseData(py::module& m)
 {
     /// Register the BaseData binding into the pybind11 system.
-    py::class_<BaseData, std::unique_ptr<sofa::core::objectmodel::BaseData, pybind11::nodelete>> data(m, "Data", sofapython3::doc::baseData::BaseDataClass);
+    //py::class_<BaseData, std::unique_ptr<sofa::core::objectmodel::BaseData, pybind11::nodelete>> data(m, "Data", sofapython3::doc::baseData::BaseDataClass);
+
+    auto data =getPythonClassForBaseData(m);
     data.def("getName", [](BaseData& b){ return b.getName(); }, sofapython3::doc::baseData::getName);
     data.def("setName", [](BaseData& b, const std::string& s){ b.setName(s); }, sofapython3::doc::baseData::setName);
     data.def("getCounter", [](BaseData& self) { return self.getCounter(); }, sofapython3::doc::baseData::getCounter);