Closed grayfall closed 4 years ago
I take my words back. It was my PR that broke things. Reordering int llt = nfeats * (nfeats - 1) / 2;
into int llt = (nfeats / 2) * (nfeats - 1);
solves the overflow issue, but it introduces a problem with integer division and rounding.
Hello again. Having resolved the overflow issue (https://github.com/tpq/propr/pull/14), I have proceeded with my work to find another bizarre bug. First of all, I'd like to reiterate that I no longer have issues with the number of features. The following code with synthetic data runs perfectly fine:
Now on to the new issue. It seems that
propr
can't handle an odd number of features. Here is an exampleHere's the traceback
The same thing happens with any odd number of features and never happens with an even number. To be clear, I'm running
propr
installed from the master branch with my PR. The PR hasn't changed any logic, so it shouldn't be at fault.