Fix CI ValueError: Unknown loss type: dapo (#4173)

albertvillanova · web-flow · commit a7b54f988bd8 · 2025-09-30T18:27:21.000+02:00
diff --git a/tests/slow/test_grpo_slow.py b/tests/slow/test_grpo_slow.py
@@ -79,6 +79,7 @@ def test_training_with_liger_grpo_loss(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
`@@ -79,6 +79,7 @@ def test_training_with_liger_grpo_loss(self, model_name):`
`79`	`79`	`max_completion_length=self.max_length,`
`80`	`80`	`report_to="none",`
`81`	`81`	`logging_strategy="no",`
	`82`	`+ loss_type="bnpo", # liger-kernel does not support "dapo" default; see https://github.com/linkedin/Liger-Kernel/issues/620`
`82`	`83`	`)`
`83`	`84`
`84`	`85`	`model = AutoModelForCausalLM.from_pretrained(model_name)`