yade-dev team mailing list archive

[noreply@xxxxxxxxxxxxx]

------------------------------------------------------------
revno: 3770
committer: Anton Gladky <gladky.anton@xxxxxxxxx>
timestamp: Thu 2016-01-07 11:50:51 +0100
message:
  Fix an order of calculation of c in SpherePack
  
  It seems, it is a bug in a new Eigen.
  Reported on their bugtracker.
  http://eigen.tuxfamily.org/bz/show_bug.cgi?id=1144
modified:
  pkg/dem/SpherePack.cpp


--
lp:yade
https://code.launchpad.net/~yade-pkg/yade/git-trunk

Your team Yade developers is subscribed to branch lp:yade.
To unsubscribe from this branch go to https://code.launchpad.net/~yade-pkg/yade/git-trunk/+edit-subscription

=== modified file 'pkg/dem/SpherePack.cpp'
--- pkg/dem/SpherePack.cpp	2016-01-07 10:47:37 +0000
+++ pkg/dem/SpherePack.cpp	2016-01-07 10:50:51 +0000
@@ -186,7 +186,7 @@
 				for(int axis=0; axis<3; axis++) {
 					c[axis]=rnd();//coordinates in [0,1]
 				}
-				c=mn + hSize*c; //coordinates in reference frame (inside the base cell)
+				c = hSize*c + mn; //coordinates in reference frame (inside the base cell)
 			}
 			
 			size_t packSize=pack.size();