Uploaded image for project: 'Commons Math'
  1. Commons Math
  2. MATH-891

SpearmansCorrelation fails when using NaturalRanking together with NaNStrategy.REMOVED

Rank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersConvert to sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.0
    • 3.2
    • None
    • None

    Description

      As reported by Martin Rosellen on the users mailinglist:

      Using a NaturalRanking with a REMOVED NaNStrategy can result in an exception when NaN are contained in the input arrays.

      The current implementation just removes the NaN values where they occur, without taken care to remove the corresponding values in the other array.

      Attachments

        1. MATH-891.patch
          8 kB
          Thomas Neidhart

        Issue Links

        relates to

        Task - A task that needs to be done. MATH-958 Remove support for NaNStrategy.REMOVED in SpearmansCorrelation

        • Major - Major loss of function.
        • Resolved

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            tn Thomas Neidhart
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment