Skip to content

Commit

Permalink
GoldfarbIdnani QP fix: i, j instead of j, i (accord-net#171)
Browse files Browse the repository at this point in the history
  • Loading branch information
Petr Částek committed Feb 1, 2016
1 parent 642427c commit 873ddbe
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1173,9 +1173,9 @@ private void qpgen2(double[,] dmat, double[] dvec, int[] iact, out int nact, ref

for (int i = 0; i < n; i++)
{
temp = dmat[i, it1 - 1];
dmat[i, it1 - 1] = dmat[i, it1];
dmat[i, it1] = temp;
temp = dmat[it1 - 1, i];
dmat[it1 - 1, i] = dmat[it1, i];
dmat[it1, i] = temp;
}
}
else
Expand Down

1 comment on commit 873ddbe

@cesarsouza
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I found out the same issue when comparing to the original Fortran code yesterday. Sorry for not committing it earlier.

Please sign in to comment.