Commit 10fd9ba
committed
[feat] add validation shuffle (verl-project#1886)
### Checklist Before Starting
- [x] Search for similar PR(s).
### What does this PR do?
In scenarios involving multiple validation sets, where the difficulty
levels of these sets differ significantly and the generated content
lengths vary notably, the order in which the validation sets are
processed can have a substantial impact on the validation speed.
### High-Level Design
add validation shuffle
### Usage Example
> Provide usage example(s) for easier usage.
```python
validation_shuffle: True
```
### Test
Validation speed increase of over 10%.
### Checklist Before Submitting
- [x] Read the [Contribute
Guide](https://github.com/volcengine/verl?tab=readme-ov-file#contribution-guide).
- [x] Apply [pre-commit
checks](https://github.com/volcengine/verl?tab=readme-ov-file#code-linting-and-formatting).
- [ ] Add `[BREAKING]` to the PR title if it breaks any API.
- [ ] Update the documentation about your changes in the
[docs](https://github.com/volcengine/verl/tree/main/docs).
- [ ] New CI unit test(s) are added to cover the code path.
- [ ] Rely on existing unit tests on CI that covers the code path.1 parent 4858ae4 commit 10fd9ba
2 files changed
+2
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| 16 | + | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
550 | 550 | | |
551 | 551 | | |
552 | 552 | | |
553 | | - | |
| 553 | + | |
554 | 554 | | |
555 | 555 | | |
556 | 556 | | |
| |||
0 commit comments