yade-dev team mailing list archive

[Jan Stránský <honzik.stransky@xxxxxxxxx>]

Hi Bruno, thanks for info, I have updated the docstring
Jan

** Changed in: yade
       Status: New => Fix Released

-- 
You received this bug notification because you are a member of Yade
developers, which is subscribed to Yade.
https://bugs.launchpad.net/bugs/1738232

Title:
  Inconsistency in doc of Body::intrs and code
  InteractionContainer::insert

Status in Yade:
  Fix Released

Bug description:
  Hello,

  in the docs of Body [1] there is

  "NOTE: (currently) does not contain all interactions with this body
  (only those where otherId>id), since performance issues with such data
  duplication have not yet been investigated."

  but in the InteractionContainer::insert, the opposite id is added to
  both bodies [2,3].

  Is the documentation wrong?

  cheers
  Jan

  [1] https://github.com/yade/trunk/blob/master/core/Body.hpp#L89
  [2] https://github.com/yade/trunk/blob/master/core/InteractionContainer.cpp#L22
  [3] https://github.com/yade/trunk/blob/master/core/InteractionContainer.cpp#L23

To manage notifications about this bug go to:
https://bugs.launchpad.net/yade/+bug/1738232/+subscriptions