verl icon indicating copy to clipboard operation
verl copied to clipboard

fix: support empty val dataset

Open shinytang6 opened this issue 6 months ago • 2 comments

Checklist Before Starting

  • [ ] Search for similar PR(s).

What does this PR do?

We have same requirements as https://github.com/volcengine/verl/issues/1676 This PR support user training with empty val dataset

High-Level Design

Demonstrate the high-level design if this PR is complex.

Specific Changes

List the specific changes.

API

Demonstrate how the API changes if any.

Usage Example

Provide usage example(s) for easier usage.

# Add code snippet or script demonstrating how to use this 

Test

For changes that can not be tested by CI (e.g., algorithm implementation, new model support), validate by experiment(s) and show results like training curve plots, evaluatuion results, etc.

Additional Info.

  • Issue Number: Fixes issue # or discussion # if any.
  • Training: [Note which backend this PR will affect: FSDP, Megatron, both, or none]
  • Inference: [Note which backend this PR will affect: vLLM, SGLang, both, or none]

Checklist Before Submitting

  • [ ] Read the Contribute Guide.
  • [ ] Apply pre-commit checks.
  • [ ] Add [BREAKING] to the PR title if it breaks any API.
  • [ ] Update the documentation about your changes in the docs.
  • [ ] Add CI test(s) if necessary.

shinytang6 avatar May 29 '25 08:05 shinytang6

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 29 '25 08:05 CLAassistant

@vermouth1992 pls help take a look, thx

shinytang6 avatar May 29 '25 09:05 shinytang6