BarkleyBG/multilevelMatching

Add check for is.factor(W)

Opened this issue · 2 comments

BarkleyBG commented

I think that when is.factor(W) is TRUE then reordering data will throw an error.

BarkleyBG commented
  • determine whether there is an error
    • Yes, i've built this error in because I haven't handled factors well
  • deteremine where error is
    • getIDs() in multiMatch()
    • temp$ix : $ operator is invalid for atomic vectors in reorderByTreatment_legacy()
  • start unit test to fix the reorderByTreatment_legacy() function for original code
  • start branch to fix the original code
  • start unit test to fix the multiMatch() implementation
  • start branch to fix the multiMatch() implementation

(also, testing @BarkleyBG emails)