kosukeimai / fastLink

R package fastLink: Fast Probabilistic Record Linkage
272 stars 48 forks source link

The documentation for return.all does not seem to match the function default? #64

Open zross opened 2 years ago

zross commented 2 years ago

I'm using v ‘0.6.0’ and unless I'm missing something I think the help incorrectly states the default for return.all as TRUE -- the function seems to have return.all = FALSE as the default.

image image
aalexandersson commented 2 years ago

Disclaimer: I am a regular fastLink user, not a fastLink developer.

Yes, and thank you for mentioning the documentation issue. The commit change was on Mar 29, 2019:https://github.com/kosukeimai/fastLink/commit/7915255f13c90292f6d4de71e3eac3bde6cb1033

tedenamorado commented 2 years ago

@zross, thanks for raising this issue! As @aalexandersson mentions, you are correct. I will update the help file.