Skip to content

Commit

Permalink
Add assert in BertModel that untie_embeddings_and_output_weights is F…
Browse files Browse the repository at this point in the history
…alse
  • Loading branch information
jaredcasper committed Apr 6, 2023
1 parent c43d19a commit 1e2a040
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions megatron/model/bert_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,9 @@ def __init__(self,
super(BertModel, self).__init__()
args = get_args()

# TODO this option is not yet implemented in BERT
assert args.untie_embeddings_and_output_weights is False

self.fp16_lm_cross_entropy = args.fp16_lm_cross_entropy
self.add_binary_head = add_binary_head
self.parallel_output = parallel_output
Expand Down

0 comments on commit 1e2a040

Please sign in to comment.