Good one, thanks. :) B Le jeu. 22 nov. 2018 10:24, Jérôme Duriez <jerome.dur...@irstea.fr> a écrit :
> Public bug reported: > > See https://answers.launchpad.net/yade/+question/676284 and: > > ##################### > O.bodies.append(sphere((0,0,0),1,dynamic=False)) > O.bodies.append(sphere((0,0,1.9),1,dynamic=False)) > > O.step() > > print 'BodyNumInteractions Histogram', > bodyNumInteractionsHistogram(aabbExtrema()) > #################### > > which returns (with trunk version) : > > ArgumentError: Python argument types in > yade._utils.bodyNumInteractionsHistogram(list) > did not match C++ signature: > bodyNumInteractionsHistogram(boost::python::tuple aabb, bool > contactOnly=False) > > > because my commit [*] changed the Python return type of aabbExtrema() from > tuple to list. > > > Possible (and easiest) fix would be to change all corresponding py::tuple > aabb from py::list aabb. Agree ? > > Or do we want this function to deal with Python tuples and not lists ?.. > > > Jérôme > > [*] > > https://github.com/yade/trunk/commit/1db13fb1183b9e294dc9761da76cfa4fc2791cc1 > > ** Affects: yade > Importance: Undecided > Status: New > > -- > You received this bug notification because you are a member of Yade > developers, which is subscribed to Yade. > https://bugs.launchpad.net/bugs/1804621 > > Title: > bodyNumInteractionsHistogram broken > > Status in Yade: > New > > Bug description: > See https://answers.launchpad.net/yade/+question/676284 and: > > ##################### > O.bodies.append(sphere((0,0,0),1,dynamic=False)) > O.bodies.append(sphere((0,0,1.9),1,dynamic=False)) > > O.step() > > print 'BodyNumInteractions Histogram', > bodyNumInteractionsHistogram(aabbExtrema()) > #################### > > which returns (with trunk version) : > > ArgumentError: Python argument types in > yade._utils.bodyNumInteractionsHistogram(list) > did not match C++ signature: > bodyNumInteractionsHistogram(boost::python::tuple aabb, bool > contactOnly=False) > > > because my commit [*] changed the Python return type of aabbExtrema() > from tuple to list. > > > Possible (and easiest) fix would be to change all corresponding > py::tuple aabb from py::list aabb. Agree ? > > Or do we want this function to deal with Python tuples and not lists > ?.. > > > Jérôme > > [*] > > https://github.com/yade/trunk/commit/1db13fb1183b9e294dc9761da76cfa4fc2791cc1 > > To manage notifications about this bug go to: > https://bugs.launchpad.net/yade/+bug/1804621/+subscriptions > > _______________________________________________ > Mailing list: https://launchpad.net/~yade-dev > Post to : yade-dev@lists.launchpad.net > Unsubscribe : https://launchpad.net/~yade-dev > More help : https://help.launchpad.net/ListHelp >
_______________________________________________ Mailing list: https://launchpad.net/~yade-dev Post to : yade-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~yade-dev More help : https://help.launchpad.net/ListHelp
