Really like this package and have used it quite a bit, so thank for putting it out there! Wanted to flag an issue that I encountered. The dynIRT function requires the bills to be in chronological order. Line 12 of getLast_dynIRT.cpp says // Key assumption here and throughout code: bills sorted chronologically before input. However, this point is never raised in the documentation, isn't given as an error message, and isn't traceable since it occurs after the Rcpp .Call(). It can't be reproduced with the example Martin and Quinn (2002) data since those bills are in chronological order --- but once you put them out of order you get an opaque indexing error. There's no way to tell that this is required unless you dig through the source code.
Obviously, this isn't that important of an issue, but I thought I'd point it out in case someone else ran into the same problem.
Really like this package and have used it quite a bit, so thank for putting it out there! Wanted to flag an issue that I encountered. The
dynIRTfunction requires the bills to be in chronological order. Line 12 ofgetLast_dynIRT.cppsays// Key assumption here and throughout code: bills sorted chronologically before input. However, this point is never raised in the documentation, isn't given as an error message, and isn't traceable since it occurs after the Rcpp.Call(). It can't be reproduced with the example Martin and Quinn (2002) data since those bills are in chronological order --- but once you put them out of order you get an opaque indexing error. There's no way to tell that this is required unless you dig through the source code.Obviously, this isn't that important of an issue, but I thought I'd point it out in case someone else ran into the same problem.