Fix DistributedDP Optimizer for Fast Gradient Clipping (#662)

Summary: Pull Request resolved: #662 The step function incorrectly called "original_optimizer.original_optimizer" instead of "original_optimizer". Fixed it now. Differential Revision: D60484128
pytorch · Aug 1, 2024 · 26b4f71 · 26b4f71
1 parent f1d0e02
commit 26b4f71
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/opacus/__init__.py b/opacus/__init__.py
@@ -15,13 +15,15 @@
 
 from . import utils
 from .grad_sample import GradSampleModule
+from .grad_sample_fast_gradient_clipping import GradSampleModuleFastGradientClipping
 from .privacy_engine import PrivacyEngine
 from .version import __version__
 
 
 __all__ = [
     "PrivacyEngine",
     "GradSampleModule",
+    "GradSampleModuleFastGradientClipping",
     "utils",
     "__version__",
 ]
diff --git a/opacus/optimizers/ddpoptimizer_fast_gradient_clipping.py b/opacus/optimizers/ddpoptimizer_fast_gradient_clipping.py
@@ -76,6 +76,6 @@ def step(
 
         if self.pre_step():
             self.reduce_gradients()
-            return self.original_optimizer.original_optimizer.step()
+            return self.original_optimizer.step()
         else:
             return None