Skip to content

Optimized differential privacy implementation#997

Merged
tharvik merged 2 commits intodevelopfrom
differential-privacy
Feb 3, 2026
Merged

Optimized differential privacy implementation#997
tharvik merged 2 commits intodevelopfrom
differential-privacy

Conversation

@ahzero7d1
Copy link
Collaborator

  1. Gaussian noise calibration implementation

    • Added Gaussian noise calibration for differential privacy (DP) based on privacy budget (epsilon), failure possibility (delta) and weight clipping radius
  2. Adaptive clipping mechanism

    • Implemented adaptive clipping radius using historical moving average of weight updates
  3. cli testing tool modification

  4. Webapp interface updates

    • Added UI fields for the new DP parameters

Copy link
Collaborator

@tharvik tharvik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the great work! I added a few comments on how to improve parts of it but the core feature is there 🎉

btw, I don't remember if we talked about adding a test for the feature. for an integration test, you can add one in server/tests/e2e/federated.spec.ts trying out the feature and if you have some hardening/units tests in mind, you can also some in discojs/src/privacy.spec.ts to ensure that it works in many conditions.

@ahzero7d1 ahzero7d1 requested a review from tharvik January 5, 2026 06:17
@tharvik tharvik force-pushed the differential-privacy branch from 136b2ce to 7fcc166 Compare February 3, 2026 16:48
Copy link
Collaborator

@tharvik tharvik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated it a bit with the latest changes for byzantine impl, thanks for the huge contributions!

@tharvik tharvik merged commit aa89f7b into develop Feb 3, 2026
15 checks passed
@tharvik tharvik deleted the differential-privacy branch February 3, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants