Skip to content

Bug in Python version (lmm.py) #17

Description

@DLCorcoran

Congrats on the publication of the package, I'm looking forward to testing it out.

While working with the Python version, I found that on line 275 of lmm.py you are doubling the p-value twice, resulting in p-values > 1:

pval = 2 * t_dist.sf(np.abs(tval), df=n - p) * 2

It seems to be correct in the R version:

pval <- 2 * pt(-abs(tval), df = n-p)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions