X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/444bae311b5e7581b17444ba7c94c28249b16cc1..91fe7904c884b2d0f5e7408d6e9d1d99886bd365:/src/bindings/python/simgrid_python.cpp diff --git a/src/bindings/python/simgrid_python.cpp b/src/bindings/python/simgrid_python.cpp index 5d990334f8..0b1721b134 100644 --- a/src/bindings/python/simgrid_python.cpp +++ b/src/bindings/python/simgrid_python.cpp @@ -24,7 +24,6 @@ #include "simgrid/kernel/ProfileBuilder.hpp" #include "simgrid/kernel/routing/NetPoint.hpp" -#include "src/kernel/context/Context.hpp" #include #include #include @@ -47,6 +46,7 @@ using simgrid::s4u::Actor; using simgrid::s4u::ActorPtr; using simgrid::s4u::Engine; using simgrid::s4u::Host; +using simgrid::s4u::Link; using simgrid::s4u::Mailbox; XBT_LOG_NEW_DEFAULT_CATEGORY(python, "python"); @@ -86,12 +86,21 @@ PYBIND11_MODULE(simgrid, m) // Internal exception used to kill actors and sweep the RAII chimney (free objects living on the stack) static py::object pyForcefulKillEx(py::register_exception(m, "ActorKilled")); + py::register_exception(m, "NetworkFailureException"); + py::register_exception(m, "TimeoutException"); + py::register_exception(m, "HostFailureException"); + py::register_exception(m, "StorageFailureException"); + py::register_exception(m, "VmFailureException"); + py::register_exception(m, "CancelException"); + /* this_actor namespace */ m.def_submodule("this_actor", "Bindings of the s4u::this_actor namespace. See the C++ documentation for details.") .def( "debug", [](const char* s) { XBT_DEBUG("%s", s); }, "Display a logging message of 'debug' priority.") .def( "info", [](const char* s) { XBT_INFO("%s", s); }, "Display a logging message of 'info' priority.") + .def( + "warning", [](const char* s) { XBT_WARN("%s", s); }, "Display a logging message of 'warning' priority.") .def( "error", [](const char* s) { XBT_ERROR("%s", s); }, "Display a logging message of 'error' priority.") .def("execute", py::overload_cast(&simgrid::s4u::this_actor::execute), @@ -115,18 +124,21 @@ PYBIND11_MODULE(simgrid, m) .def("exit", &simgrid::s4u::this_actor::exit, py::call_guard(), "kill the current actor") .def( "on_exit", - [](py::object fun) { + [](py::object cb) { + py::function fun = py::reinterpret_borrow(cb); fun.inc_ref(); // FIXME: why is this needed for tests like actor-kill and actor-lifetime? - simgrid::s4u::this_actor::on_exit([fun](bool /*failed*/) { + simgrid::s4u::this_actor::on_exit([fun](bool failed) { + py::gil_scoped_acquire py_context; // need a new context for callback try { - py::gil_scoped_acquire py_context; // need a new context for callback - fun(); + fun(failed); } catch (const py::error_already_set& e) { xbt_die("Error while executing the on_exit lambda: %s", e.what()); } }); }, - py::call_guard(), "") + py::call_guard(), + "Define a lambda to be called when the actor ends. It takes a bool parameter indicating whether the actor " + "was killed. If False, the actor finished peacefully.") .def("get_pid", &simgrid::s4u::this_actor::get_pid, "Retrieves PID of the current actor") .def("get_ppid", &simgrid::s4u::this_actor::get_ppid, "Retrieves PPID of the current actor (i.e., the PID of its parent)."); @@ -141,20 +153,56 @@ PYBIND11_MODULE(simgrid, m) return new simgrid::s4u::Engine(&argc, argv.data()); }), "The constructor should take the parameters from the command line, as is ") - .def_static("get_clock", &Engine::get_clock, - "The simulation time, ie the amount of simulated seconds since the simulation start.") - .def_static( - "instance", []() { return Engine::get_instance(); }, "Retrieve the simulation engine") - .def("get_all_hosts", &Engine::get_all_hosts, "Returns the list of all hosts found in the platform") - .def("get_all_links", &Engine::get_all_links, "Returns the list of all links found in the platform") - - .def("get_netzone_root", &Engine::get_netzone_root, "Retrieve the root netzone, containing all others.") - .def("get_all_netpoints", &Engine::get_all_netpoints) - .def("get_netzone_root", &Engine::get_netzone_root) + .def_static("get_clock", + []() // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx( + PyExc_DeprecationWarning, + "get_clock() is deprecated and will be dropped after v3.33, use `Engine.clock` instead.", 1); + return Engine::get_clock(); + }) + .def_property_readonly_static( + "clock", [](py::object /* self */) { return Engine::get_clock(); }, + "The simulation time, ie the amount of simulated seconds since the simulation start.") + .def_property_readonly_static( + "instance", [](py::object /* self */) { return Engine::get_instance(); }, "Retrieve the simulation engine") + .def("get_all_hosts", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_all_hosts() is deprecated and will be dropped after v3.33, use all_hosts instead.", 1); + return self.attr("all_hosts"); + }) + .def_property_readonly("all_hosts", &Engine::get_all_hosts, "Returns the list of all hosts found in the platform") + .def("get_all_links", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_all_links() is deprecated and will be dropped after v3.33, use all_links instead.", 1); + return self.attr("all_links"); + }) + .def_property_readonly("all_links", &Engine::get_all_links, "Returns the list of all links found in the platform") + .def("get_all_netpoints", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx( + PyExc_DeprecationWarning, + "get_all_netpoints() is deprecated and will be dropped after v3.33, use all_netpoints instead.", 1); + return self.attr("all_netpoints"); + }) + .def_property_readonly("all_netpoints", &Engine::get_all_netpoints) + .def("get_netzone_root", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_netzone_root() is deprecated and will be dropped after v3.3, use netzone_root instead.", + 1); + return self.attr("netzone_root"); + }) + .def_property_readonly("netzone_root", &Engine::get_netzone_root, + "Retrieve the root netzone, containing all others.") .def("netpoint_by_name", &Engine::netpoint_by_name_or_null) .def("netzone_by_name", &Engine::netzone_by_name_or_null) - .def("set_netzone_root", &Engine::set_netzone_root) - .def("load_platform", &Engine::load_platform, "Load a platform file describing the environment") .def("load_deployment", &Engine::load_deployment, "Load a deployment file and launch the actors that it contains") .def("run", &Engine::run, py::call_guard(), "Run the simulation until its end") @@ -165,8 +213,8 @@ PYBIND11_MODULE(simgrid, m) "register_actor", [](Engine* e, const std::string& name, py::object fun_or_class) { e->register_actor(name, [fun_or_class](std::vector args) { + py::gil_scoped_acquire py_context; try { - py::gil_scoped_acquire py_context; /* Convert the std::vector into a py::tuple */ py::tuple params(args.size() - 1); for (size_t i = 1; i < args.size(); i++) @@ -236,10 +284,19 @@ PYBIND11_MODULE(simgrid, m) .def("create_router", &simgrid::s4u::NetZone::create_router, "Create a router") .def("set_parent", &simgrid::s4u::NetZone::set_parent, "Set the parent of this zone") .def("set_property", &simgrid::s4u::NetZone::set_property, "Add a property to this zone") - .def("get_netpoint", &simgrid::s4u::NetZone::get_netpoint, "Retrieve the netpoint associated to this zone") + .def("get_netpoint", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_netpoint() is deprecated and will be dropped after v3.33, use netpoint instead.", 1); + return self.attr("netpoint"); + }) + .def_property_readonly("netpoint", &simgrid::s4u::NetZone::get_netpoint, + "Retrieve the netpoint associated to this zone") .def("seal", &simgrid::s4u::NetZone::seal, "Seal this NetZone") .def_property_readonly( - "name", [](const simgrid::s4u::NetZone* self) { return self->get_name(); }, "The name of this network zone"); + "name", [](const simgrid::s4u::NetZone* self) { return self->get_name(); }, + "The name of this network zone (read-only property)."); /* Class ClusterCallbacks */ py::class_(m, "ClusterCallbacks", "Callbacks used to create cluster zones") @@ -260,11 +317,11 @@ PYBIND11_MODULE(simgrid, m) /* Class Host */ py::class_> host( m, "Host", "Simulated host. See the C++ documentation for details."); - host.def("by_name", &Host::by_name, "Retrieves a host from its name, or die") + host.def_static("by_name", &Host::by_name, py::arg("name"), "Retrieves a host from its name, or die") .def( "route_to", - [](simgrid::s4u::Host* h, simgrid::s4u::Host* to) { - auto* list = new std::vector(); + [](const simgrid::s4u::Host* h, const simgrid::s4u::Host* to) { + auto* list = new std::vector(); double bw = 0; h->route_to(to, *list, &bw); return make_tuple(list, bw); @@ -306,12 +363,49 @@ PYBIND11_MODULE(simgrid, m) " \"\"\"\n\n" "The second function parameter is the periodicity: the time to wait after the last event to start again over " "the list. Set it to -1 to not loop over.") - .def("get_pstate_count", &Host::get_pstate_count, "Retrieve the count of defined pstate levels") - .def("get_pstate_speed", &Host::get_pstate_speed, "Retrieve the maximal speed at the given pstate") - .def("get_netpoint", &Host::get_netpoint, "Retrieve the netpoint associated to this host") + .def("get_pstate_count", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx( + PyExc_DeprecationWarning, + "get_pstate_count() is deprecated and will be dropped after v3.33, use pstate_count instead.", 1); + return self.attr("pstate_count"); + }) + .def_property_readonly("pstate_count", &Host::get_pstate_count, "Retrieve the count of defined pstate levels") + .def("get_pstate_speed", + [](py::object self, int state) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx( + PyExc_DeprecationWarning, + "get_pstate_speed() is deprecated and will be dropped after v3.33, use pstate_speed instead.", 1); + return self.attr("pstate_speed")(state); + }) + .def("pstate_speed", &Host::get_pstate_speed, py::call_guard(), + "Retrieve the maximal speed at the given pstate") + .def("get_netpoint", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_netpoint() is deprecated and will be dropped after v3.33, use netpoint instead.", 1); + return self.attr("netpoint"); + }) + .def_property_readonly("netpoint", &Host::get_netpoint, "Retrieve the netpoint associated to this zone") .def("get_disks", &Host::get_disks, "Retrieve the list of disks in this host") - .def("set_core_count", &Host::set_core_count, py::call_guard(), - "Set the number of cores in the CPU") + .def("set_core_count", + [](py::object self, double count) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "set_core_count() is deprecated and will be dropped after v3.33, use core_count instead.", + 1); + self.attr("core_count")(count); + }) + .def_property( + "core_count", &Host::get_core_count, + [](Host* h, int count) { + py::gil_scoped_release gil_guard; + return h->set_core_count(count); + }, + "Manage the number of cores in the CPU") .def("set_coordinates", &Host::set_coordinates, py::call_guard(), "Set the coordinates of this host") .def("set_sharing_policy", &simgrid::s4u::Host::set_sharing_policy, py::call_guard(), @@ -328,26 +422,41 @@ PYBIND11_MODULE(simgrid, m) py::gil_scoped_release gil_guard; h->set_pstate(i); }, - "The current pstate") - .def("current", &Host::current, py::call_guard(), + "The current pstate (read/write property).") + .def_static("current", &Host::current, py::call_guard(), "Retrieves the host on which the running actor is located.") .def_property_readonly( "name", [](const Host* self) { return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC }, - "The name of this host") + "The name of this host (read-only property).") .def_property_readonly("load", &Host::get_load, "Returns the current computation load (in flops per second), NOT taking the external load " - "into account. This is the currently achieved speed.") + "into account. This is the currently achieved speed (read-only property).") .def_property_readonly( "speed", &Host::get_speed, "The peak computing speed in flops/s at the current pstate, NOT taking the external load into account. " - "This is the max potential speed.") - .def_property_readonly( - "available_speed", &Host::get_available_speed, - "Get the available speed ratio, between 0 and 1.\n" - "This accounts for external load (see :py:func:`set_speed_profile() `)."); + "This is the max potential speed (read-only property).") + .def_property_readonly("available_speed", &Host::get_available_speed, + "Get the available speed ratio, between 0 and 1.\n" + "This accounts for external load (see :py:func:`set_speed_profile() " + "`) (read-only property).") + .def_static( + "on_creation_cb", + [](py::object cb) { + Host::on_creation_cb([cb](Host& h) { + py::function fun = py::reinterpret_borrow(cb); + py::gil_scoped_acquire py_context; // need a new context for callback + try { + fun(&h); + } catch (const py::error_already_set& e) { + xbt_die("Error while executing the on_creation lambda : %s", e.what()); + } + }); + }, + py::call_guard(), ""); + py::enum_(host, "SharingPolicy") .value("NONLINEAR", simgrid::s4u::Host::SharingPolicy::NONLINEAR) .value("LINEAR", simgrid::s4u::Host::SharingPolicy::LINEAR) @@ -369,7 +478,8 @@ PYBIND11_MODULE(simgrid, m) py::arg("cb") = simgrid::s4u::NonLinearResourceCb()) .def("seal", &simgrid::s4u::Disk::seal, py::call_guard(), "Seal this disk") .def_property_readonly( - "name", [](const simgrid::s4u::Disk* self) { return self->get_name(); }, "The name of this disk"); + "name", [](const simgrid::s4u::Disk* self) { return self->get_name(); }, + "The name of this disk (read-only property)."); py::enum_(disk, "SharingPolicy") .value("NONLINEAR", simgrid::s4u::Disk::SharingPolicy::NONLINEAR) .value("LINEAR", simgrid::s4u::Disk::SharingPolicy::LINEAR) @@ -385,19 +495,19 @@ PYBIND11_MODULE(simgrid, m) netpoint(m, "NetPoint", "NetPoint object"); /* Class Link */ - py::class_> link( - m, "Link", "Network link. See the C++ documentation for details."); - link.def("set_latency", py::overload_cast(&simgrid::s4u::Link::set_latency), + py::class_> link(m, "Link", + "Network link. See the C++ documentation for details."); + link.def("set_latency", py::overload_cast(&Link::set_latency), py::call_guard(), "Set the latency as a string. Accepts values with units, such as ‘1s’ or ‘7ms’.\nFull list of accepted " "units: w (week), d (day), h, s, ms, us, ns, ps.") - .def("set_latency", py::overload_cast(&simgrid::s4u::Link::set_latency), - py::call_guard(), "Set the latency as a float (in seconds).") - .def("set_bandwidth", &simgrid::s4u::Link::set_bandwidth, py::call_guard(), + .def("set_latency", py::overload_cast(&Link::set_latency), py::call_guard(), + "Set the latency as a float (in seconds).") + .def("set_bandwidth", &Link::set_bandwidth, py::call_guard(), "Set the bandwidth (in byte per second).") .def( "set_bandwidth_profile", - [](simgrid::s4u::Link* l, const std::string& profile, double period) { + [](Link* l, const std::string& profile, double period) { l->set_bandwidth_profile(simgrid::kernel::profile::ProfileBuilder::from_string("", profile, period)); }, py::call_guard(), @@ -416,7 +526,7 @@ PYBIND11_MODULE(simgrid, m) "the list. Set it to -1 to not loop over.") .def( "set_latency_profile", - [](simgrid::s4u::Link* l, const std::string& profile, double period) { + [](Link* l, const std::string& profile, double period) { l->set_latency_profile(simgrid::kernel::profile::ProfileBuilder::from_string("", profile, period)); }, py::call_guard(), @@ -435,7 +545,7 @@ PYBIND11_MODULE(simgrid, m) "the list. Set it to -1 to not loop over.") .def( "set_state_profile", - [](simgrid::s4u::Link* l, const std::string& profile, double period) { + [](Link* l, const std::string& profile, double period) { l->set_state_profile(simgrid::kernel::profile::ProfileBuilder::from_string("", profile, period)); }, "Specify a profile modeling the churn. " @@ -450,39 +560,40 @@ PYBIND11_MODULE(simgrid, m) "The second function parameter is the periodicity: the time to wait after the last event to start again over " "the list. Set it to -1 to not loop over.") - .def("turn_on", &simgrid::s4u::Link::turn_on, py::call_guard(), "Turns the link on.") - .def("turn_off", &simgrid::s4u::Link::turn_off, py::call_guard(), "Turns the link off.") - .def("is_on", &simgrid::s4u::Link::is_on, "Check whether the link is on.") + .def("turn_on", &Link::turn_on, py::call_guard(), "Turns the link on.") + .def("turn_off", &Link::turn_off, py::call_guard(), "Turns the link off.") + .def("is_on", &Link::is_on, "Check whether the link is on.") - .def("set_sharing_policy", &simgrid::s4u::Link::set_sharing_policy, py::call_guard(), + .def("set_sharing_policy", &Link::set_sharing_policy, py::call_guard(), "Set sharing policy for this link") - .def("set_concurrency_limit", &simgrid::s4u::Link::set_concurrency_limit, - py::call_guard(), "Set concurrency limit for this link") - .def("set_host_wifi_rate", &simgrid::s4u::Link::set_host_wifi_rate, py::call_guard(), + .def("set_concurrency_limit", &Link::set_concurrency_limit, py::call_guard(), + "Set concurrency limit for this link") + .def("set_host_wifi_rate", &Link::set_host_wifi_rate, py::call_guard(), "Set level of communication speed of given host on this Wi-Fi link") - .def("by_name", &simgrid::s4u::Link::by_name, "Retrieves a Link from its name, or dies") - .def("seal", &simgrid::s4u::Link::seal, py::call_guard(), "Seal this link") + .def_static("by_name", &Link::by_name, "Retrieves a Link from its name, or dies") + .def("seal", &Link::seal, py::call_guard(), "Seal this link") .def_property_readonly( "name", - [](const simgrid::s4u::Link* self) { + [](const Link* self) { return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC }, "The name of this link") - .def_property_readonly("bandwidth", &simgrid::s4u::Link::get_bandwidth, "The bandwidth (in bytes per second)") - .def_property_readonly("latency", &simgrid::s4u::Link::get_latency, "The latency (in seconds)"); - - py::enum_(link, "SharingPolicy") - .value("NONLINEAR", simgrid::s4u::Link::SharingPolicy::NONLINEAR) - .value("WIFI", simgrid::s4u::Link::SharingPolicy::WIFI) - .value("SPLITDUPLEX", simgrid::s4u::Link::SharingPolicy::SPLITDUPLEX) - .value("SHARED", simgrid::s4u::Link::SharingPolicy::SHARED) - .value("FATPIPE", simgrid::s4u::Link::SharingPolicy::FATPIPE) + .def_property_readonly("bandwidth", &Link::get_bandwidth, + "The bandwidth (in bytes per second) (read-only property).") + .def_property_readonly("latency", &Link::get_latency, "The latency (in seconds) (read-only property)."); + + py::enum_(link, "SharingPolicy") + .value("NONLINEAR", Link::SharingPolicy::NONLINEAR) + .value("WIFI", Link::SharingPolicy::WIFI) + .value("SPLITDUPLEX", Link::SharingPolicy::SPLITDUPLEX) + .value("SHARED", Link::SharingPolicy::SHARED) + .value("FATPIPE", Link::SharingPolicy::FATPIPE) .export_values(); /* Class LinkInRoute */ py::class_ linkinroute(m, "LinkInRoute", "Abstraction to add link in routes"); - linkinroute.def(py::init()); - linkinroute.def(py::init()); + linkinroute.def(py::init()); + linkinroute.def(py::init()); py::enum_(linkinroute, "Direction") .value("UP", simgrid::s4u::LinkInRoute::Direction::UP) .value("DOWN", simgrid::s4u::LinkInRoute::Direction::DOWN) @@ -490,11 +601,24 @@ PYBIND11_MODULE(simgrid, m) .export_values(); /* Class Split-Duplex Link */ - py::class_>(m, "SplitDuplexLink", - "Network split-duplex link") - .def("get_link_up", &simgrid::s4u::SplitDuplexLink::get_link_up, "Get link direction up") - .def("get_link_down", &simgrid::s4u::SplitDuplexLink::get_link_down, "Get link direction down"); + py::class_>( + m, "SplitDuplexLink", "Network split-duplex link") + .def("get_link_up", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_link_up() is deprecated and will be dropped after v3.33, use link_up instead.", 1); + return self.attr("link_up"); + }) + .def_property_readonly("link_up", &simgrid::s4u::SplitDuplexLink::get_link_up, "Get link direction up") + .def("get_link_down", + [](py::object self) // XBT_ATTRIB_DEPRECATED_v334 + { + PyErr_WarnEx(PyExc_DeprecationWarning, + "get_link_down() is deprecated and will be dropped after v3.33, use link_down instead.", 1); + return self.attr("link_down"); + }) + .def_property_readonly("link_down", &simgrid::s4u::SplitDuplexLink::get_link_down, "Get link direction down"); /* Class Mailbox */ py::class_>( @@ -502,13 +626,23 @@ PYBIND11_MODULE(simgrid, m) .def( "__str__", [](const Mailbox* self) { return std::string("Mailbox(") + self->get_cname() + ")"; }, "Textual representation of the Mailbox`") - .def("by_name", &Mailbox::by_name, py::call_guard(), "Retrieve a Mailbox from its name") + .def_static("by_name", &Mailbox::by_name, + py::call_guard(), + py::arg("name"), + "Retrieve a Mailbox from its name") .def_property_readonly( "name", [](const Mailbox* self) { return std::string(self->get_name().c_str()); // Convert from xbt::string because of MC }, - "The name of that mailbox") + "The name of that mailbox (read-only property).") + .def( + "put", + [](Mailbox* self, py::object data, int size, double timeout) { + data.inc_ref(); + self->put(data.ptr(), size, timeout); + }, + py::call_guard(), "Blocking data transmission with a timeout") .def( "put", [](Mailbox* self, py::object data, int size) { @@ -523,10 +657,18 @@ PYBIND11_MODULE(simgrid, m) return self->put_async(data.ptr(), size); }, py::call_guard(), "Non-blocking data transmission") + .def( + "put_init", + [](Mailbox* self, py::object data, int size) { + data.inc_ref(); + return self->put_init(data.ptr(), size); + }, + py::call_guard(), + "Creates (but don’t start) a data transmission to that mailbox.") .def( "get", [](Mailbox* self) { - py::object data = pybind11::reinterpret_steal(self->get()); + py::object data = py::reinterpret_steal(self->get()); // data.dec_ref(); // FIXME: why does it break python-actor-create? return data; }, @@ -558,14 +700,40 @@ PYBIND11_MODULE(simgrid, m) "Test whether the communication is terminated.") .def("wait", &simgrid::s4u::Comm::wait, py::call_guard(), "Block until the completion of that communication.") + .def("wait_for", &simgrid::s4u::Comm::wait_for, + py::arg("timeout"), + py::call_guard(), + "Block until the completion of that communication, or raises TimeoutException after the specified timeout.") + .def("detach", [](simgrid::s4u::Comm* self) { + return self->detach(); + }, + py::return_value_policy::reference_internal, + py::call_guard(), + "Start the comm, and ignore its result. It can be completely forgotten after that.") // use py::overload_cast for wait_all/wait_any, until the overload marked XBT_ATTRIB_DEPRECATED_v332 is removed .def_static( "wait_all", py::overload_cast&>(&simgrid::s4u::Comm::wait_all), - py::call_guard(), "Block until the completion of all communications in the list.") + py::arg("comms"), + py::call_guard(), + "Block until the completion of all communications in the list.") + .def_static("wait_all_for", &simgrid::s4u::Comm::wait_all_for, + py::arg("comms"), py::arg("timeout"), + py::call_guard(), + "Block until the completion of all communications in the list, or raises TimeoutException after " + "the specified timeout.") .def_static( "wait_any", py::overload_cast&>(&simgrid::s4u::Comm::wait_any), + py::arg("comms"), + py::call_guard(), + "Block until the completion of any communication in the list and return the index of the terminated one.") + .def_static( + "wait_any_for", + py::overload_cast&, double>(&simgrid::s4u::Comm::wait_any_for), + py::arg("comms"), py::arg("timeout"), py::call_guard(), - "Block until the completion of any communication in the list and return the index of the terminated one."); + "Block until the completion of any communication in the list and return the index of the terminated " + "one, or -1 if a timeout occurred." + ); /* Class Io */ py::class_(m, "Io", "I/O activities. See the C++ documentation for details.") @@ -587,7 +755,7 @@ PYBIND11_MODULE(simgrid, m) py::gil_scoped_release gil_guard; return self->get_remaining(); }, - "Amount of flops that remain to be computed until completion.") + "Amount of flops that remain to be computed until completion (read-only property).") .def_property_readonly( "remaining_ratio", [](simgrid::s4u::ExecPtr self) { @@ -595,9 +763,10 @@ PYBIND11_MODULE(simgrid, m) return self->get_remaining_ratio(); }, "Amount of work remaining until completion from 0 (completely done) to 1 (nothing done " - "yet).") + "yet) (read-only property).") .def_property("host", &simgrid::s4u::Exec::get_host, &simgrid::s4u::Exec::set_host, - "Host on which this execution runs. Only the first host is returned for parallel executions.") + "Host on which this execution runs. Only the first host is returned for parallel executions. " + "Changing this value migrates the execution.") .def("test", &simgrid::s4u::Exec::test, py::call_guard(), "Test whether the execution is terminated.") .def("cancel", &simgrid::s4u::Exec::cancel, py::call_guard(), "Cancel that execution.") @@ -615,8 +784,8 @@ PYBIND11_MODULE(simgrid, m) fun.inc_ref(); // FIXME: why is this needed for tests like exec-async, exec-dvfs and exec-remote? args.inc_ref(); // FIXME: why is this needed for tests like actor-migrate? return simgrid::s4u::Actor::create(name, h, [fun, args]() { + py::gil_scoped_acquire py_context; try { - py::gil_scoped_acquire py_context; fun(*args); } catch (const py::error_already_set& ex) { if (ex.matches(pyForcefulKillEx)) { @@ -627,19 +796,27 @@ PYBIND11_MODULE(simgrid, m) } }); }, - py::call_guard(), "Create an actor from a function or an object. See the :ref:`example `.") + py::call_guard(), + "Create an actor from a function or an object. See the :ref:`example `.") .def_property( "host", &Actor::get_host, [](Actor* a, Host* h) { py::gil_scoped_release gil_guard; a->set_host(h); }, - "The host on which this actor is located") - .def_property_readonly("name", &Actor::get_cname, "The name of this actor.") - .def_property_readonly("pid", &Actor::get_pid, "The PID (unique identifier) of this actor.") + "The host on which this actor is located. Changing this value migrates the actor.\n\n" + "If the actor is currently blocked on an execution activity, the activity is also migrated to the new host. " + "If it’s blocked on another kind of activity, an error is raised as the mandated code is not written yet. " + "Please report that bug if you need it.\n\n" + "Asynchronous activities started by the actor are not migrated automatically, so you have to take care of " + "this yourself (only you knows which ones should be migrated). ") + .def_property_readonly("name", &Actor::get_cname, "The name of this actor (read-only property).") + .def_property_readonly("pid", &Actor::get_pid, "The PID (unique identifier) of this actor (read-only property).") .def_property_readonly("ppid", &Actor::get_ppid, - "The PID (unique identifier) of the actor that created this one.") - .def("by_pid", &Actor::by_pid, "Retrieve an actor by its PID") + "The PID (unique identifier) of the actor that created this one (read-only property).") + .def_static("by_pid", &Actor::by_pid, py::arg("pid"), "Retrieve an actor by its PID") + .def("set_auto_restart", &Actor::set_auto_restart, py::call_guard(), + "Specify whether the actor shall restart when its host reboots.") .def("daemonize", &Actor::daemonize, py::call_guard(), "This actor will be automatically terminated when the last non-daemon actor finishes (more info in the C++ " "documentation).") @@ -649,11 +826,11 @@ PYBIND11_MODULE(simgrid, m) .def("join", py::overload_cast(&Actor::join, py::const_), py::call_guard(), "Wait for the actor to finish (more info in the C++ documentation).", py::arg("timeout")) .def("kill", &Actor::kill, py::call_guard(), "Kill that actor") - .def("kill_all", &Actor::kill_all, py::call_guard(), "Kill all actors but the caller.") .def("self", &Actor::self, "Retrieves the current actor.") .def("is_suspended", &Actor::is_suspended, "Returns True if that actor is currently suspended.") .def("suspend", &Actor::suspend, py::call_guard(), "Suspend that actor, that is blocked until resume()ed by another actor.") .def("resume", &Actor::resume, py::call_guard(), - "Resume that actor, that was previously suspend()ed."); + "Resume that actor, that was previously suspend()ed.") + .def_static("kill_all", &Actor::kill_all, py::call_guard(), "Kill all actors but the caller."); }