Dirk,

See attached for minor edits related to Walker Alias method.  Based on
the timing tests from the gallery draft and discussion with Johnathan,
we decided it was a bad idea to include code that deviates from R's
behavior, and is slower than R to boot.  Thus, the C++ sample now
throws an informative error under the appropriate conditions.  I
updated the unitTests accordingly (and cleaned them up a bit).

Incidentally, there's now a stub in place for Walker Alias method
sample function, as well as an associated unitTest that's now
commented out.
-Christian

Attachment: walker.throw.diff
Description: Binary data

_______________________________________________
Rcpp-devel mailing list
Rcpp-devel@lists.r-forge.r-project.org
https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel

Reply via email to