Menu

#17 Pairwise Distribution/Summary are inadequately tested

open-accepted
Gaurav
5
2006-05-11
2006-05-08
Gaurav
No

We need more testing for this code, as it has been
changing fairly erratically between versions. Testing
had been kept aside earlier as the code was extremely
old, mouldy, and in much need for repair. Since 0.9.1.
1, I've replaced PairwiseDistribution with nice, clean
code, which is a LOT less painful to deal with.
However, we need extensive testing to make sure that:
1. Numbers changing from last time are becoming
INCREASINGLY accurate, not less so.
2. They're not going to change *again* in the future
(except by becoming less accurate, which should be
protected by incorporating said tests into the Test
framework).

Discussion

  • Gaurav

    Gaurav - 2006-05-11
    • status: open --> open-accepted
     
  • Gaurav

    Gaurav - 2006-05-11

    Logged In: YES
    user_id=712450

    Test 1: Ran a dataset (see attachment below). Replaced all
    gaps with 'A's to prevent any overlap issues (I'll also run
    another dataset from Guanyang tomorrow). I tested the
    following:
    1. Number of distances (based on the species composition
    from SpeciesSummary): expected = obtained from export from
    PairwiseSummary
    2. Binning of distances: identical.
    3. Overlap values (NOT size of overlap, but just the limits)
    for both overlap and 5% cutoffs: identical.

    I'll do test 2 (with Guanyang's dataset) tomorrow.

     

Log in to post a comment.

MongoDB Logo MongoDB
Gen AI apps are built with MongoDB Atlas
Atlas offers built-in vector search and global availability across 125+ regions. Start building AI apps faster, all in one place.