Skip to content

Update the default theta parameter definition in LM & LMTR and remove νInv #178

@MaxenceGollier

Description

@MaxenceGollier

Currently, we compute the steps in LM and LMTR as νInv = (1 + θ) * (σmax^2 + σk) in LM and νInv = (1 + θ) * σmax^2 in LMTR.
It would be more readable (and actually more compliant with the article notation to have respectively νInv = (σmax^2 + σk)/θ and νInv = (σmax^2/θ instead.

I will open a PR to change the default value of θ accordingly and do these modifications. @dpo It is related to a comment you made in #172 .
@MohamedLaghdafHABIBOULLAH fyi.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    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