search

jackolney / CareCascade

CareCascade Model Repository
0 stars 0 forks source link

OutputIndexIssue #39

Closed jackolney closed 9 years ago

jackolney commented 9 years ago 

// I believe this to be wrong.
while(theQ->GetTime() > yr[i] && i<27)
    foo++;

// If GetTime() == yr[i] then you could run into problems...
// For instance, if HIV testing becomes available in 2004.
// If GetTime() == 01/01/04 then i still thinks it is 2004
// So it will appear as if people are getting HIV tests early, when in fact they are not.
// The index flicks over from 02/01/04 but tracks people on the 1st January getting HIV-tests (small minority)

/* FIX */
while(theQ->GetTime() >= yr[i] && i<27)
    foo++;

Testing this fix in WP19 branch: 6d3929e

jackolney commented 9 years ago

Fixed in 76b2eaf