Skip to content

Commit

Permalink
chore: remove todo comment from test
Browse files Browse the repository at this point in the history
  • Loading branch information
jolars committed Dec 6, 2023
1 parent f856ef4 commit a0d7f45
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion tests/standardize.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ computeMeanAndStdDev(const Eigen::SparseMatrix<double>& x)

for (int j = 0; j < p; ++j) {
x_means(j) = x.col(j).sum() / n;
// TODO: Reconsider this implementation since it might overflow.
x_stddevs(j) =
std::sqrt(x.col(j).squaredNorm() / n - std::pow(x_means(j), 2));
}
Expand Down

0 comments on commit a0d7f45

Please sign in to comment.