Some code is matching on partial variable names

[briandk]

For example:

stats.vcj<-jitter(stats.vc, am = ammt)

source: https://github.com/briandk/granovaGG/blob/dev/R/granovagg.1w.R#L244

[WilDoane]

Since "am" is the actual parameter being passed into jitter, an R {base} function, which is being partially matched on the formal parameter "amount", can't we simply change

stats.vcj<-jitter(stats.vc, am = ammt)

to

stats.vcj<-jitter(stats.vc, ammount = ammt)

with zero impact on other code and the public interface?

[briandk]

In short, yes.

Part of what's frustrating is that that particular statement doesn't get used by any of our function's computations. It's now inconsequential legacy code that I've been too timid to take out.

[briandk]

I think I'm going to address this issue in our next major release (1.3, presumably), and keep a pretty conservative list of changes for the 1.2 release.