← Back to team overview

yade-dev team mailing list archive

Re: [Bug 1476159] [NEW] several ForceContainer.addF behaves non-intuitively with permanent=True

 

You are right. I agree with your solution.
However I suggest to make addF(id,(2,0,0),True) return error/warning
with explicit message (the function changed, please use that one, etc.)
instead of just standard python error. It will make transition easier
for those who use the current version.
Bruno

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

Title:
  several ForceContainer.addF behaves non-intuitively with
  permanent=True

Status in Yade:
  New

Bug description:
  Hi all,

  consider

  O.forces.addF(id,(2,0,0),True)
  ... # some code, a few steps or whatever
  O.forces.addF(id,(4,0,0),True)

  what should be the result? Intuitively (according to addF name), I
  would expect applied force (6,0,0). However, the permanent force is
  rewritten with (4,0,0), so applied force (4,0,0) is the result..

  I suggest to split addF to 2 cases:
  1) addF without permanent parameter
  2) setPermanentForce ()or something similar), substituting addF(permanent=True), with the name reflecting more accurately what it really does..

  any suggestions or comments?
  cheers
  Jan

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


References