Skip to content

BUG: PrecisionMvNormalRV missing factor of 1/2 for log determinant in logp function #7751

Open
@ckrapu

Description

@ckrapu
Contributor

Describe the issue:

At this line,

logp = -0.5 * (k * pt.log(2 * np.pi) + quadratic_form) + logdet

should be

logp = -0.5 * (k * pt.log(2 * np.pi) + quadratic_form + logdet)

Reproduceable code example:

N/A

Error message:

PyMC version information:

5.21.1

Context for the issue:

No response

Activity

ricardoV94

ricardoV94 commented on Apr 9, 2025

@ricardoV94
Member

@ckrapu wanna open a PR by any chance?

shanegladson

shanegladson commented on Apr 28, 2025

@shanegladson

@ckrapu wanna open a PR by any chance?

This issue seems inactive so I'm happy to take it over if available

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Participants

      @fonnesbeck@ckrapu@shanegladson@ricardoV94

      Issue actions

        BUG: PrecisionMvNormalRV missing factor of 1/2 for log determinant in logp function · Issue #7751 · pymc-devs/pymc