Fix CI slow test ValueError: Unknown loss type: dapo (#4254)

albertvillanova · web-flow · commit 98488e0946c0 · 2025-10-10T16:37:02.000+02:00
diff --git a/tests/slow/test_grpo_slow.py b/tests/slow/test_grpo_slow.py
@@ -118,6 +118,7 @@ def test_training_with_liger_grpo_loss_and_peft(self, model_name):
             max_completion_length=self.max_length,
             report_to="none",
             logging_strategy="no",
+            loss_type="bnpo",  # liger-kernel does not support "dapo" default; see https://github.com/linkedin/Liger-Kernel/issues/620
         )
 
         model = AutoModelForCausalLM.from_pretrained(model_name)

Original file line number	Diff line number	Diff line change
`@@ -118,6 +118,7 @@ def test_training_with_liger_grpo_loss_and_peft(self, model_name):`
`118`	`118`	`max_completion_length=self.max_length,`
`119`	`119`	`report_to="none",`
`120`	`120`	`logging_strategy="no",`
	`121`	`+ loss_type="bnpo", # liger-kernel does not support "dapo" default; see https://github.com/linkedin/Liger-Kernel/issues/620`
`121`	`122`	`)`
`122`	`123`
`123`	`124`	`model = AutoModelForCausalLM.from_pretrained(model_name)`