Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-23305

Replace Fraction of DIAObjects Updated metric

    XMLWordPrintable

    Details

      Description

      The fraction of DIAObjects updated metric is fragile to execution order and has proven hard to understand in practice. This ticket is to replace it with N_associated/N_DIAsources which is similar in nature but more straightforward to compute.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            Sorry for the late question, but what does "N_associated" mean? If it's the number of sources that are matched to an existing DIAObject, that is also sensitive to execution order (in particular, it is 0 for the first visit). If it's the number of sources assigned to any DIAObject, that makes the metric identically 1 with the current algorithm.

            Show
            krzys Krzysztof Findeisen added a comment - - edited Sorry for the late question, but what does "N_associated" mean? If it's the number of sources that are matched to an existing DIAObject, that is also sensitive to execution order (in particular, it is 0 for the first visit). If it's the number of sources assigned to any DIAObject, that makes the metric identically 1 with the current algorithm.
            Hide
            ebellm Eric Bellm added a comment -

            Good point, Krzysztof Findeisen. Since we're now running in Gen3 in a reproducible order, let's mark this as WON'T FIX for now and determine later if we should drop the metric entirely.

            Show
            ebellm Eric Bellm added a comment - Good point, Krzysztof Findeisen . Since we're now running in Gen3 in a reproducible order, let's mark this as WON'T FIX for now and determine later if we should drop the metric entirely.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              ebellm Eric Bellm
              Watchers:
              Eric Bellm, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: