Skip to content

Adding option to remove outliers#973

Open
nucleosynthesis wants to merge 2 commits intomainfrom
nckw_add_option_to_remove_outliers
Open

Adding option to remove outliers#973
nucleosynthesis wants to merge 2 commits intomainfrom
nckw_add_option_to_remove_outliers

Conversation

@nucleosynthesis
Copy link
Copy Markdown
Contributor

Added option in HybridNew to remove outliers from test-stat distributions before calculating p-values/limits.

This option removes test-statistic values from the test-statistic distributions that are further than $X\sigma$ from the mean of the distribution, where $\sigma$ is the standard deviation of the distribution and $X$ is the value passed to the option, before calculating $p-$values and limits. Setting this to some reasonably large number (eg 50) should remove the spurious values. By default, no removal of outliers is performed.

Added text to documentation about this option.

Added option in `HybridNew` to remove outliers from test-stat distributions before calculating p-values/limits.

This option removes test-statistic values from the test-statistic distributions that are further than $X\sigma$ from the mean of the distribution, where $\sigma$ is the standard deviation of the distribution and $X$ is the value passed to the option, before calculating $p-$values and limits. Setting this to some reasonably large number (eg 50) should remove the spurious values. By default, no removal of outliers is performed.

Added text to documentation about this option.
@nucleosynthesis
Copy link
Copy Markdown
Contributor Author

Obviously would be better to calculate median rather than mean before outlier removal, will update PR with that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant