Closed nygge closed 12 years ago
One issue with this one, regarding the copyright, I don't think I can merge this giving the copyright to you without a CLA signed by you. If you don't care I can leave my copyright in there and attribute the patch to you in commit message. Let me know.
Thats fine.
Anders
On Wed, Nov 30, 2011 at 8:55 PM, Joe Williams reply@reply.github.com wrote:
One issue with this one, regarding the copyright, I don't think I can merge this giving the copyright to you without a CLA signed by you. If you don't care I can leave my copyright in there and attribute the patch to you in commit message. Let me know.
Reply to this email directly or view it on GitHub: https://github.com/boundary/folsom/pull/10#issuecomment-2970186
Started a branch with this and the other pull request,
merged
Rewrite statistics to avoid repeated list traversals. Also implement Spearman correlation that actually works.