Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • evalhyd-cpp evalhyd-cpp
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

La forge institutionnelle d'INRAE étant en production depuis le 10 juin 2025, nous vous invitons à y créer vos nouveaux projets.

  • HYCAR-HydroHYCAR-Hydro
  • evalhydevalhyd
  • evalhyd-cppevalhyd-cpp
  • Issues
  • #5
Closed
Open
Issue created Apr 13, 2023 by Thibault Hallouin@thibault.hallouinOwner

fix spearman rank correlation calculation for KGENP and KGENP_D

As mentioned in https://github.com/xtensor-stack/xtensor/issues/2677, when the observations and/or predictions contain ties, the default (and fastest) C++ algorithm (std::sort) does not guarantee the order of equal elements, which results in different values for the spearman rank correlation component of the non-parametric version of KGE.

To fix this, the C++ algorithm (std::stable_sort) needs to be used instead, so that the order of equal elements is guaranteed. I have proposed a fix via a PR to xtensor (https://github.com/xtensor-stack/xtensor/pull/2681) that is awaiting approval. Until then, KGENP and KGENP_D will not be available (31cf8da5).

Edited Apr 13, 2023 by Thibault Hallouin
Assignee
Assign to
Time tracking