Fix to column name encoding issue

richarddmorey / BayesFactor

BayesFactor R package for Bayesian data analysis with common statistical models.

https://richarddmorey.github.io/BayesFactor/

132 stars 49 forks source link

Fix to column name encoding issue #132

Closed jonathon-love closed 5 years ago

jonathon-love commented 5 years ago

Fixes #129

richarddmorey commented 5 years ago

So this just gets rid of the Base64 encoding stuff? What implications does this have for BayesFactor, JASP, and jamovi?

jonathon-love commented 5 years ago

so it gets rid of the base64 stuff, but it keeps the variable escaping (composeTerm()). i would say that this base64 stuff was required to work-around a bug in model.Matrix() and is no longer necessary.

i don't think either jasp or jamovi access the model matrix by column name (or at all?), and if they did, we would have found out about this issue earlier.