Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
python: fix register_function when passed a class
[simgrid.git] / src / bindings / python / simgrid_python.cpp
index 52387a6..28300bc 100644 (file)
@@ -1,3 +1,8 @@
+/* Copyright (c) 2018. The SimGrid Team. All rights reserved.          */
+
+/* This program is free software; you can redistribute it and/or modify it
+ * under the terms of the license (GNU LGPL) which comes with this package. */
+
 #include <functional>
 #include <memory>
 #include <string>
@@ -14,6 +19,7 @@
 #include <simgrid/s4u/Actor.hpp>
 #include <simgrid/s4u/Engine.hpp>
 #include <simgrid/s4u/Host.hpp>
+#include <simgrid/s4u/Mailbox.hpp>
 
 #include <boost/intrusive_ptr.hpp>
 
@@ -22,6 +28,7 @@ using simgrid::s4u::Actor;
 using simgrid::s4u::ActorPtr;
 using simgrid::s4u::Engine;
 using simgrid::s4u::Host;
+using simgrid::s4u::Mailbox;
 
 XBT_LOG_NEW_DEFAULT_CATEGORY(python, "python");
 
@@ -73,29 +80,72 @@ PYBIND11_MODULE(simgrid, m)
       .def("load_deployment", &Engine::load_deployment,
           "Load a deployment file and launch the actors that it contains, see :cpp:func:`simgrid::s4u::Engine::load_deployment()`")
       .def("run", &Engine::run, "Run the simulation")
-      .def("register_function", [](Engine*, std::string name, std::function<void(std::vector<std::string>)> f) {
+      .def("register_function", [](Engine*, std::string name, py::object obj) {
         simgrid::simix::register_function(name,
-            [f](std::vector<std::string> args) -> simgrid::simix::ActorCode {
-          return [args, f]() { f(args); };
+            [obj](std::vector<std::string> args) -> simgrid::simix::ActorCode {
+          return [obj, args]() {
+            /* Convert the std::vector into a py::tuple */
+            py::tuple params(args.size()-1);
+            for (size_t i=1; i<args.size(); i++)
+              params[i-1] = py::cast(args[i]);
+
+            PyObject *result = PyObject_CallObject(obj.ptr(), params.ptr());
+            if (!result)
+                throw pybind11::error_already_set();
+
+            /* If I was passed a class, I just built an instance, so I need to call it now */
+            if (PyCallable_Check(result)) {
+              py::object obj2 = pybind11::reinterpret_steal<py::object>(pybind11::handle(static_cast<PyObject*>(result)));
+              obj2();
+            }
+          };
         });
-      }, "Registers the main function of an actor that will be launched from the deployment file, , see :cpp:func:`simgrid::s4u::Engine::register_function()`");
+      }, "Registers the main function of an actor, see :cpp:func:`simgrid::s4u::Engine::register_function()`")
+      ;
 
   /* Class Host */
   py::class_<simgrid::s4u::Host, std::unique_ptr<Host, py::nodelete>>(m, "Host", "Simulation Engine, see :ref:`class s4u::Host <API_s4u_Host>`").def(
       "by_name", &Host::by_name, "Retrieve a host from its name, or die");
 
-  /* Class Actor */
-  // Select the right template instantiation
-  simgrid::s4u::ActorPtr (*create_actor)(std::string, Host*, std::function<void()>) = &Actor::create;
+  /* Class Mailbox */
+  py::class_<simgrid::s4u::Mailbox, std::unique_ptr<Mailbox, py::nodelete>>(m, "Mailbox", "Mailbox, see :ref:`class s4u::Mailbox <API_s4u_Mailbox>`")
+      .def("by_name", &Mailbox::by_name, "Retrieve a Mailbox from its name, see :cpp:func:`simgrid::s4u::Mailbox::by_name()`")
+      .def("get_name", &Mailbox::get_name, "Retrieves the name of that host, see :cpp:func:`simgrid::s4u::Mailbox::get_name()`")
+      .def("put", [](Mailbox self, py::object data, int size) {
+        data.inc_ref();
+        self.put(data.ptr(), size);
+      }, "Blocking data transmission, see :cpp:func:`void simgrid::s4u::Mailbox::put(void*, uint64_t)`")
+      .def("get", [](Mailbox self) -> py::object {
+         py::object data = pybind11::reinterpret_steal<py::object>(pybind11::handle(static_cast<PyObject*>(self.get())));
+         data.dec_ref();
+         return data;
+      }, "Blocking data reception, see :cpp:func:`void* simgrid::s4u::Mailbox::get()`");
 
+  /* Class Actor */
   py::class_<simgrid::s4u::Actor, ActorPtr>(m, "Actor", ""
       "An actor is an independent stream of execution in your distributed application, see :ref:`class s4u::Actor <API_s4u_Actor>`")
-    .def("create", create_actor, "Create an actor from a function, see :cpp:func:`simgrid::s4u::Actor::create()`")
-    .def("create", [](std::string name, Host* host) -> std::function<ActorPtr(std::function<void()>)> {
-      return [name, host](std::function<void()> f) -> ActorPtr {
-        return simgrid::s4u::Actor::create(name, host, std::move(f));
-      };
-    }, "Create an actor from a functor");
+
+    .def("create", [](py::args args, py::kwargs kwargs) {
+        xbt_assert(args.size()>2, "Creating an actor takes at least 3 parameters: name, host, and main function.");
+        return simgrid::s4u::Actor::create(args[0].cast<std::string>(), args[1].cast<Host*>(), [args]() {
+          py::tuple funargs(args.size()-3);
+          for (size_t i=3; i<args.size(); i++)
+            funargs[i-3] = args[i];
+
+          PyObject *result = PyObject_CallObject(args[2].ptr(), funargs.ptr());
+          if (!result)
+              throw pybind11::error_already_set();
+        });
+    }, "Create an actor from a function or an object, see :cpp:func:`simgrid::s4u::Actor::create()`")
+    /*
+    .def("create", [](std::string name, Host* host, py::object obj) -> ActorPtr {
+        xbt_assert(pybind11::hasattr(obj, "__call__"), "Your object does not implement the __call__() method");
+
+        return simgrid::s4u::Actor::create(name, host, [obj](){
+          obj.attr("__call__")();
+        });
+    }, "Create an actor from a python object")
+    */;
 
 
 }