Skip to content

Commit 5a829b0

Browse files
fix(network): reset_parameters checks nonexistent (+3 more)
1 parent 060b857 commit 5a829b0

1 file changed

Lines changed: 16 additions & 4 deletions

File tree

fastgen/networks/cosmos_predict2/network.py

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -431,6 +431,11 @@ def forward(
431431
- [output, features]: If feature_indices is non-empty
432432
- (output, logvar): If return_logvar=True
433433
"""
434+
if feature_indices is None:
435+
feature_indices = set()
436+
if return_features_early and len(feature_indices) == 0:
437+
return []
438+
434439
x_B_T_H_W_D, rope_emb_L_1_1_D, extra_pos_emb = self.prepare_embedded_sequence(
435440
x_B_C_T_H_W,
436441
fps=fps,
@@ -641,9 +646,16 @@ def encode(self, text: Union[str, List[str]], precision: torch.dtype = torch.bfl
641646
input_ids_batch.append(formatted)
642647

643648
input_ids = torch.cat(input_ids_batch, dim=0).to(self.device)
649+
pad_token_id = self.tokenizer.pad_token_id or 0
650+
attention_mask = (input_ids != pad_token_id).long()
644651

645652
# Forward pass with hidden states
646-
outputs = self.model(input_ids=input_ids, output_hidden_states=True, return_dict=True)
653+
outputs = self.model(
654+
input_ids=input_ids,
655+
attention_mask=attention_mask,
656+
output_hidden_states=True,
657+
return_dict=True,
658+
)
647659
hidden_states = outputs.hidden_states # Tuple of (num_layers + 1) tensors
648660

649661
# Skip embedding layer (index 0), normalize and combine layers
@@ -1007,8 +1019,8 @@ def reset_parameters(self):
10071019
nn.init.zeros_(m.bias)
10081020

10091021
# Reinitialize RoPE buffers (non-persistent buffers must be recomputed)
1010-
if hasattr(self.transformer, "rope_embedder") and self.transformer.rope_embedder is not None:
1011-
rope = self.transformer.rope_embedder
1022+
if hasattr(self.transformer, "pos_embedder") and self.transformer.pos_embedder is not None:
1023+
rope = self.transformer.pos_embedder
10121024
device = next(rope.buffers()).device
10131025

10141026
# Recompute the sequence buffer
@@ -1327,7 +1339,7 @@ def forward(
13271339
- ([output, features], logvar): If both features and logvar requested
13281340
"""
13291341
if feature_indices is None:
1330-
feature_indices = {}
1342+
feature_indices = set()
13311343
if return_features_early and len(feature_indices) == 0:
13321344
# Exit immediately if user requested this.
13331345
return []

0 commit comments

Comments
 (0)