Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Potential FPE suppression/hiding in GsfUtils.hpp #4031

Open
andiwand opened this issue Jan 15, 2025 · 1 comment
Open

Potential FPE suppression/hiding in GsfUtils.hpp #4031

andiwand opened this issue Jan 15, 2025 · 1 comment
Labels

Comments

@andiwand
Copy link
Contributor

// If something is not finite here, just leave the weight as it is
if (std::isfinite(factor)) {
weights.at(tip) *= factor;
}

This looks like silent FPE treatment which might either come from the chi2 being too large pushing detR to inf or caused by detR being negative in case the covariance is already broken. Both should in principle not happen but we protect against it and silence any occurrence right now.

In #3449 I observed an FPE at

const auto factor = std::sqrt(1. / detR) * safeExp(-0.5 * chi2);
which indicates that detR is negative and this should only happen if at least one of the covariances is broken.

Copy link

This issue/PR has been automatically marked as stale because it has not had recent activity. The stale label will be removed if any interaction occurs.

@github-actions github-actions bot added the Stale label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant