cloudyr/aws.comprehend

detect_entities fails when more than 1 entity by document

Closed this issue · 1 comments

To reproduce:

txt <-c("Amazon provides web services, like Google.",
            "Jeff is their leader.")
detect_entities(txt)

#>  Error in data.frame(..., check.names = FALSE) : 
#>  arguments imply differing number of rows: 2, 3 

This is because of an incorrect handling of the index in a call to cbind. I am just writing a detect_syntax function which had the same issue. I'll raise a PR. There is already a proposed fix (#2) but since I'll also PR my detect_syntax function, we might as well keep the codebase consistent.

Duplicate of #2