Igh input validation - Githubissues

globusharris commented 5 years ago

Added input validation checks.

A few notes:

Covariance has issues marked in it, which I am working on in a separate branch.
I did a cursory merge of the input validation into the glm statistic but did not bother to do fixes in objective mech that will allow input validation to run (checkRange input validation is run within censorData), since objective mech is not current priority.

globusharris commented 5 years ago

This resolves issues #99, #100, #101

globusharris commented 5 years ago

This resolves issue #99, #100.

globusharris commented 5 years ago

I'm unable to build this because of an issue in vignettes/dp-glm.Rmd (though this is not changed in this PR as far as I can tell). Seems like things work when I delete that file.

Oh that actually makes sense, because I left out refactoring the objective mechanism with the changes in checkRange, since there are so many other issues with the objective mechanism that it seemed silly to do so. I hadn't thought about how that would result in the vignette not building, since we've mostly been completely ignoring that vignette. I'll push a work-around so that we can keep the vignette (at least for now, it probably shouldn't be included in any release that doesn't also include the obj mech fix).

privacytoolsproject / PSI-Library

Igh input validation #104