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.

Sign up for GitHub

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

Jump to bottom

1.0 API #273

Merged
merged 262 commits into from
Nov 29, 2021
Merged

1.0 API #273

merged 262 commits into from
Nov 29, 2021

Conversation

ffuuugor
Copy link
Contributor

We're introducing new Opacus API. See Readme and updated turorials for details.

For code reviews and discussions see PR history https://github.com/pytorch/opacus/pulls?q=is%3Apr+is%3Aclosed+base%3Aexperimental_v1.0

Alexandre Sablayrolles and others added 30 commits September 30, 2021 15:10
Add abstract scheduler
9a98c51
Add more schedulers. Add tests
ba1581f
lint
2048a6e
Merge pull request #212 from ffuuugor/main
67b74a0 
[new API] unit tests back to green
Fix tests and hint
cd4d74c
Remove DPOptimizer type check
cff8bdb
Merge pull request #213 from pytorch/scheduler
10cd605 
[new API] Scheduler
Basic implementation of RDP accountant.
dfc822c
Add accountant parent class
843ad30
Small fixes. Move get_noise_multiplier to rdp
2d5646b
Merge pull request #217 from pytorch/accountant
52bfa94 
Accountants
virtual step naive
fca05a5
cleanup
277fc3b
merge
7394e5f
Add secure mode
6220633
new approach for grad accumulation
eda44d3
cleanup
9c9d253
lint
a78f88c
multiprocessing support, style changes
fee0d4c
clean
9d4fae5
@karthikprasad
add module validators
5836196
@karthikprasad
add init
a60b019
Few fixes
d51e052
Merge pull request #219 from pytorch/ffuuugor_virtual_step
74726c6 
[New API] Virtual step
Add noise test to PrivacyEngine
9a42d65
@karthikprasad
address some of the review comments
8c136d0
@karthikprasad
change default behaviour from fix_and_validate to validate
de6890c
@karthikprasad
add support for custom validators in decorators
e4b6ae0
@karthikprasad
rebase and resolve merge conflicts
10fb1bd
@karthikprasad
address comment add validate to GradSampleModule
d49c970
@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@karthikprasad
Merge pull request #290 from karthikprasad/val_tut
9db24b6 
add module validator tutorial
@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@ffuuugor ffuuugor merged commit 2f4b546 into main Nov 29, 2021
@karthikprasad
Copy link
Contributor

FTR: #293, #294, #295, #296, and #298 are also part of this release despite landing after this PR.

@karthikprasad karthikprasad deleted the experimental_v1.0 branch December 1, 2021 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karthikprasad karthikprasad karthikprasad approved these changes

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ffuuugor @facebook-github-bot @karthikprasad @pierrestock @ashkan-software