Update Qwen3.5 validation sample count - #475
Conversation
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
|
WG meeting discussion 9/3/26:
|
|
+1 for the 251 samples issue. Or could we add 5 extra/duplicate samples to be as 256? |
The 0.69 target accuracy requires solving 177+ problems when the validation size was 256, and now with 251 requires solving 174+. Since the 5 removed environments were not solvable, a previously passing implementation would still be able to solve 177+, but now out of 251, which corresponds to an accuracy of 0.705. We kept the target accuracy and validation start step the same, and since 0.705 > 0.69 the convergence still happens at the same step. We didn't re-run an RCP study without late eval, so I can't say what happens earlier.
@RissyRan @ShriyaRishab 251 is the size of the validation dataset and is used for inference/generation only, it never passes through the trainer. Inference is not batched so there is no practical reason it needs to be power-of-2 - agent turns are never synchronized and may come at any time, and have different durations. Do you know of any RL frameworks that have a limitation on validation dataset size? If not then there's no reason to impose a stricter requirement to the dataset size. |
Summary
eval_samplesto equal 251 instead of 256Rationale
Validation of the reference dataset removed five invalid validation tasks. The qualified validation dataset now contains 251 samples, so compliant benchmark logs report
eval_samples: 251. The compliance checker must use the validated dataset size.The corresponding dataset update is tracked in NVIDIA-NeMo/RL#3601.
Verification
eval_sampleschecks require exactly 251git diff --check