cld2

cld2 copied to clipboard

Hi pycld2 team! Just wanted to let you know that @quinnanya used the [pycld2 binding ](https://github.com/aboSamoor/pycld2)to detect language (see [her csv](https://gist.github.com/quinnanya/d7db8a001ba0e3dca5fb7db53cdc5d9c)) on the same Conference Titles for which I used...

silviaegt

Hi there! Thank you so much for implementing this wrapper! It works so nicely on R. I was just wondering if there was anything we could do to improve its...

silviaegt

Update cld2.R

3

Old name _multi was still used in doc. Replaced by new name _mixed.

moodymudskipper

https://github.com/ropensci/cld2/blob/a5c619b24cee180090c0c5f3d493108b37853f1e/src/wrapper.cpp#L55 similar to #6, `reliabal` should be `reliable` While each of these is a breaking change, it's also confusing when code doesn't work because you spelled the word correctly.

brooksambrose

https://github.com/ropensci/cld2/blob/c8489911dd411cde0f3cde1ed102cf07e37e4cd2/R/cld2.R#L50 Should be `classification`.

brooksambrose

I'm not sure if this is a problem with the R package or the underlying library. For some URLs, detect_language (and detect_language_multi) consistently cause a segfault. ```> library(cld2) > detect_language(...

josswright

For clarity

giocomai