Skip to content

Commit 5923cb2

Browse files
committed
lint(grpo): silence pylint false positive on TrainStateNNX kwarg
pylint resolves train_state_nnx via the stale non-editable site-packages copy where TrainStateNNX.__init__ predates reference_model, so the keyword call in init_state_fn trips E1123. The local class at src/maxtext/common/train_state_nnx.py does accept reference_model. Disable the rule on this single call.
1 parent 2549adc commit 5923cb2

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

src/maxtext/experimental/rl/grpo_trainer.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -798,6 +798,7 @@ def init_state_fn():
798798
optimizer = nnx.Optimizer(nnx_model, tx, wrt=nnx.Param)
799799
# Reference uses the same init seed so it starts identical to the policy.
800800
reference_model = _create_model_partial()
801+
# pylint: disable-next=unexpected-keyword-arg
801802
return train_state_nnx.TrainStateNNX(nnx_model, optimizer, reference_model=reference_model)
802803

803804
else:

0 commit comments

Comments
 (0)