Skip to content

Commit 94d2e05

Browse files
Copilotnjzjz
andauthored
fix(test): add strict parameter to zip() calls in test_adamuon.py (#221)
* Initial plan * fix(test): add strict=True parameter to zip() calls in test_adamuon.py Co-authored-by: njzjz <9496702+njzjz@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: njzjz <9496702+njzjz@users.noreply.github.com>
1 parent f82cffe commit 94d2e05

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

source/tests/pt/test_adamuon.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,9 @@ def test_optimizer_step_smoke(self) -> None:
9797
optimizer.step()
9898

9999
# Verify all parameters with gradients changed
100-
for i, (p, init_p) in enumerate(zip(model.parameters(), initial_params)):
100+
for i, (p, init_p) in enumerate(
101+
zip(model.parameters(), initial_params, strict=True)
102+
):
101103
if p.grad is not None:
102104
self.assertFalse(
103105
torch.allclose(p, init_p),
@@ -369,7 +371,7 @@ def test_state_dict_save_load(self) -> None:
369371
params1 = list(optimizer.param_groups[0]["params"])
370372
params2 = list(optimizer2.param_groups[0]["params"])
371373

372-
for p1, p2 in zip(params1, params2):
374+
for p1, p2 in zip(params1, params2, strict=True):
373375
s1 = optimizer.state[p1]
374376
s2 = optimizer2.state[p2]
375377
self.assertEqual(set(s1.keys()), set(s2.keys()))

0 commit comments

Comments
 (0)