Addressing boolean RAPPOR decode failures when TRUE is ~ 0.

ananthr commented 8 years ago

New function .DecodeBoolean outputs an estimate for TRUE directly from EstimateBloomCounts; bypassing maps and LASSO.

This fixes a few errors that arise when the underlying distribution for TRUE is very close to zero and maps + LASSO fails to recover "a candidate solution."

Updated decode.R with a new private function .DecodeBoolean to decode when inputs are boolean
Updated decode_test.R with tests to check .DecodeBoolean
decode_assoc.R now does not use the Other category when analyzing boolean rappor inputs (k==1)
decode_test.R helper functions now output a little more information when test fails.

ilyamironov commented 8 years ago

Since this change is introduced to address a specific bug, it'd be a good idea to add a corresponding test to decode_test.R.

andychu commented 8 years ago

LGTM

OK thanks for fixing this, go ahead and merge and I can update the pipeline.

google / rappor

Addressing boolean RAPPOR decode failures when TRUE is ~ 0. #56