-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Looseness in analytic Gaussian mechanism? #39
Comments
Thanks for the issue, Ryan! 'dp_bank.get_logdelta_ana_gaussian' has two arguments. the first should be sigma. the second should be the desired epsilon. In your implementation, it seems that you passed eps first, then delta? by the way, |
Thanks for the quick response @yuxiangw, makes sense now! I'm still seeing some discrepancies for large epsilon, however:
|
hmmmm... sorry it took me a long while to get back to you on this. I don't yet know the root cause of this, but it might be due the stopping condition for the numerical computation. Yuqing or I will try to look into this, but before that... did you manage to figure out the issue? Do you want to contribute a fix to make it more stable for large epsilons? |
Here's a minimal example to demonstrate the issue:
I expect the delta = 1e-6, but it is nearly 20X larger according to DP bank.
The text was updated successfully, but these errors were encountered: