tokenizers icon indicating copy to clipboard operation
tokenizers copied to clipboard

Low-level parallelism with RcppParallel

Open lmullen opened this issue 8 years ago • 3 comments

Just as in quanteda, the functions which call Rcpp versions should be parallelized with RcppParallel. They should all have an argument that sets the number of cores: cores = getOption("mc.cores") probably defaulting to 2 if that option is not set.

lmullen avatar Apr 25 '17 21:04 lmullen

Want me to take this on?

Ironholds avatar Apr 30 '17 03:04 Ironholds

If you're willing, that would be great.

On Sat, Apr 29, 2017 at 11:02 PM Oliver Keyes [email protected] wrote:

Want me to take this on?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ropensci/tokenizers/issues/51#issuecomment-298207891, or mute the thread https://github.com/notifications/unsubscribe-auth/AALNeBFtSR1ilmqR4BR3FqEPwlvbWv4Lks5r0_nXgaJpZM4NIEBG .

-- Lincoln Mullen Assistant Professor, Department of History & Art History George Mason University

lmullen avatar May 01 '17 12:05 lmullen

Update: I have (finally) found the time to work on this but have run into an issue convincing RVector<String> objects that as<std::string> is an operation that can be performed on its members. So, this is fun?

Ironholds avatar Jan 20 '18 06:01 Ironholds