Skip to content
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

updated requirements file and readme of Privacy Meter workflow #1295

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

bandatarunkumar
Copy link
Contributor

@bandatarunkumar bandatarunkumar commented Jan 22, 2025

In this PR I've focused on fixing the requirements.txt for Privacy Meter and readme for running the privacy meter workflow, ran the privacy meter workflow with two optimizers(SGD and Adam) for 2 and 10 rounds. The workflow is running as expected without errors. The max CPU consumed while running was 18.8 gb(out of which 3.6 was previously consumed by vscode and other processes), so added note in readme to have atleast 18GB dedicated ram for running the privacy meter experiment.

@bandatarunkumar bandatarunkumar force-pushed the privacy_meter_dependancies_update branch from 6a6ec99 to 6408831 Compare January 23, 2025 06:46
@bandatarunkumar bandatarunkumar force-pushed the privacy_meter_dependancies_update branch from 6408831 to 565d20b Compare January 23, 2025 07:20
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