Skip to content

Euclidean Distance Omits sqrt(). #138

Open
Open
Euclidean Distance Omits sqrt().#138
Labels
Opinionated RealitySurprise Features (not bugs) where JGAAP differs from norms

Description

@jnoecker
jnoecker
opened on Mar 18, 2026

Euclidean Distance returns L2-squared. Affects scores, but not rankings

Activity

  1. jnoecker
    added
    Opinionated RealitySurprise Features (not bugs) where JGAAP differs from norms
    on Mar 18, 2026

  2. mvryan commented on Mar 18, 2026

    @mvryan
    mvryan
    on Mar 18, 2026
    Member

    This was an explicit design choice to reduce run time.

  3. jnoecker commented on Mar 18, 2026

    @jnoecker
    jnoecker
    on Mar 18, 2026
    MemberAuthor

    I know, that's why I didn't just "fix" it like the actual bugs. This one is just Opinionated Reality.

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

    Opinionated RealitySurprise Features (not bugs) where JGAAP differs from norms

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions