Os Keyes

digest should be in depends, not suggests, and imported

1

If it's in "suggests", it's perfectly possible to install the package with all the dependencies it claims to have, call a function, and then be told that you don't have...

Why the lowercasing?

9

Is the lowercasing of input strings in the output needed for the processing, or more a convenience/normalisation thing? My question is because it makes it hard to substitute in replacements...

Integrate methods?

8

Would it be fun if I integrated the methods so people could generate their own datasets?

Integrate bro logs

Stick a license in the GitHub repo

15

The repo doesn't have a license. Suggest putting one in? CC-0 or CC-BY-SA 3.0 or MIT or...whatever.

Rely on deques or similar

10

The push_back() style operations around vectors are vulnerable to slowdowns and reallocations since vectors are stored in contiguous memory. I'd strongly suggest using (say) deques instead, unless memory is a...

Include it. This is basically just the same as a (simplified) version of the sessionise function

Last CRAN fix

2

CRAN wants all the sprintfs changed to snprintf, but I'm about as good at C as I am at defusing semtex. Drew, any chance you'd be willing to make this...