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

Negative p-values in VT results #15

Open
abought opened this issue Jun 10, 2019 · 0 comments
Open

Negative p-values in VT results #15

abought opened this issue Jun 10, 2019 · 0 comments

Comments

@abought
Copy link
Member

abought commented Jun 10, 2019

Reported by @gjmzajac :

The variable-threshold test in Raremetal is outputting a few hundred negative p-values. Is this a known bug?
For an example, see line 912 in the p-value column here is -302.613.

@welchr notes:

My guess is you're hitting what appeared to be a rarely reached piece of code in VT when the multivariate normal distribution function didn't work, and they had implemented some kind of alternative method. That code can return negative p-values in some cases.

pvalue = CalculateMVTPvalue(score, cov_T, t_max);

First step would probably be to debug RAREMETAL and confirm where it is going wrong. I can do that if you would like, or I can help get you setup to do it, either way.

Greg has indicated he will produce a test case for this bug once other projects are done, and this ticket exists to track the known issue for other users in the interim.

@abought abought added the bug label Jul 11, 2019
@welchr welchr added type: bug and removed bug labels Mar 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants