yade-dev team mailing list archive

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

Public bug reported:

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

** 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/1738232

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

Status in Yade:
  New

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