ropensci / taxize

A taxonomic toolbelt for R
https://docs.ropensci.org/taxize
Other
264 stars 58 forks source link

gnr_resolve breaks when there is multibyte characters #904

Open zachary-foster opened 1 year ago

zachary-foster commented 1 year ago

For example:

library(taxize)
gnr_resolve("Euphorbia \xd7 lomi")
#> Error in substring(x, 2): invalid multibyte string at '<d7> lomi'

Created on 2022-11-30 with reprex v2.0.2

Can be identified before hand using validEnd. Would be nice to just delete those characters and proceed with a warning

salix-d commented 1 year ago

You can encode the input to UTF-8 with enc2utf8().

> taxize::gnr_resolve(enc2utf8("Euphorbia \xd7 lomi"))
# A tibble: 762 × 5
   user_supplied_name  submitted_name      matched_name                   data_source_title           score
 * <chr>               <chr>               <chr>                          <chr>                       <dbl>
 1 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia L.                   Catalogue of Life Checklist  0.75
 2 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia                      Wikispecies                  0.75
 3 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Chamaesyce     Wikispecies                  0.75
 4 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Alectoroctonum Wikispecies                  0.75
 5 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Euphorbia      Wikispecies                  0.75
 6 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Monadenium     Wikispecies                  0.75
 7 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Cubanthus      Wikispecies                  0.75
 8 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Lathyris       Wikispecies                  0.75
 9 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Lagascae       Wikispecies                  0.75
10 Euphorbia <d7> lomi Euphorbia <d7> lomi Euphorbia sect. Holophyllum    Wikispecies                  0.75
# … with 752 more rows
# ℹ Use `print(n = ...)` to see more rows
zachary-foster commented 1 year ago

Thanks for the tip!