search

kosukeimai / FindIt

R package FindIt
GNU General Public License v2.0
14 stars 2 forks source link

plot.PredictFindIt with locator is confusing #8

Closed HeidiSeibold closed 6 years ago

HeidiSeibold commented 6 years ago

It is very unusal that a plot function uses identify(). I feel like it would be more user-friendly to have this as an option but not as the default.

I am a frequent R user and have never seen a behaviour like this in a plot. It took me a while to figure out what had happened when the R code was stuck.

naoki-egami commented 6 years ago

Thanks for letting us know. An interactive plot is now an option rather than the default!