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. Reviewed By: HuanyuZhang Differential Revision: D60484128 fbshipit-source-id: 1bde00292b2afccc31803ebefb2c361dc7e9bb77
pytorch · Aug 2, 2024 · eb94674 · eb94674
1 parent 4804a51
commit eb94674
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/opacus/__init__.py b/opacus/__init__.py
@@ -14,14 +14,15 @@
 # limitations under the License.
 
 from . import utils
-from .grad_sample import GradSampleModule
+from .grad_sample import GradSampleModule, 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