Skip to content

Commit

Permalink
Merge branch 'ac_defaults' of github.com:NVIDIA/NeMo into ac_defaults
Browse files Browse the repository at this point in the history
  • Loading branch information
aklife97 committed Sep 9, 2023
2 parents 8c6ab5b + 08e55a3 commit 6e36202
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions nemo/collections/nlp/modules/common/megatron/transformer.py
Original file line number Diff line number Diff line change
Expand Up @@ -988,7 +988,7 @@ def __init__(
raise ValueError(
f'activations_checkpoint_method should be "uniform" or "block" when using granularity selective.'
)
self.activations_checkpoint_num_layers = num_layers # forcing all layers
self.activations_checkpoint_num_layers = num_layers # forcing all layers
elif self.activations_checkpoint_granularity == 'full':
if self.activations_checkpoint_method in ['uniform', 'block']:
if not self.activations_checkpoint_num_layers:
Expand All @@ -998,7 +998,7 @@ def __init__(
f'Got: {self.activations_checkpoint_num_layers}. Setting to 1 by default.'
)
)
self.activations_checkpoint_num_layers = 1 # keeping the old default
self.activations_checkpoint_num_layers = 1 # keeping the old default
else:
raise ValueError(
f'activations_checkpoint_method should be "uniform" or "block" when using granularity full.'
Expand Down

0 comments on commit 6e36202

Please sign in to comment.