I am not sure what I did. But removing Guava Abstract iterator actually sped up the dot, cosine, euclidean by another 60%. Things are now 2x faster than trunk. While also correcting the behavior (I hope)
https://docs.google.com/spreadsheet/ccc?key=0AhewTD_ZgznddGFQbWJCQTZXSnFULUYzdURfWDRJQlE#gid=1 Robin Anil | Software Engineer | +1 312 869 2602 | Google Inc. On Sun, Apr 14, 2013 at 8:56 PM, Robin Anil <[email protected]> wrote: > Also note that this is code gen, I have to create Element$keyType$Value > for each and every combination not just int double. and also update all > callers to user ElementIntDouble instead of Element. Is it worth it ? > > Robin Anil | Software Engineer | +1 312 869 2602 | Google Inc. > > > On Sun, Apr 14, 2013 at 8:46 PM, Ted Dunning <[email protected]>wrote: > >> Collections (no longer colt collections) are now part of mahout math. No >> need to keep them separate. The lower iterator can reference >> Vector.Element >> >> >> On Sun, Apr 14, 2013 at 6:24 PM, Robin Anil <[email protected]> wrote: >> >> > I would have loved to but Element is a sub interface in Vector. If we >> want >> > to keep colt collections separate we have to keep this separation. >> > >> > >
