Closed lmullen closed 8 years ago
From rOpenSci onboarding:
https://github.com/lmullen/tokenizers/blob/d240cddbb1d91146b3a30b90f6fc25abd6919edf/src/skip_ngrams.cpp#L9: If you want to support long vectors in R, you likely want to return and use R_xlen_t for indexing.
IMHO we are safe and should close this issue. Can't imagine the case when it can be useful to use skip_grams > 10
skip_grams > 10
From rOpenSci onboarding: