Skip to content

score reasoning only but completed responses as zero for lmarena_v3 - #3097

Merged
bxyu-nvidia merged 2 commits into
mainfrom
jveronvialard/arena-reasoning-only-scoring
Sep 6, 2026
Merged

bxyu-nvidia merged 2 commits into
mainfrom
jveronvialard/arena-reasoning-only-scoring

Conversation

@jveronvialard

Copy link
Copy Markdown
Contributor

What does this PR do?

score rollouts with policy responses marked as completed but containing only reasoning content as zero, instead of treating them as failures and excluding them from lmarena_v3 score calculation

Checklist

  • I have read the contributing guidelines.
  • The change is focused; unrelated "drive-by" edits are tracked as separate issues/PRs.
  • Tests added or updated and pass locally, or N/A for docs-only / non-code changes (so CI unit/server checks pass when applicable).
  • Pre-commit checks pass locally (pre-commit run --all-files) (so CI lint/format/copyright pass).
  • All commits have DCO sign-off (git commit -s) (so the DCO check passes).

Signed-off-by: Julien Veron Vialard <jveronvialar@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bxyu-nvidia

Copy link
Copy Markdown
Contributor

/ok to test dfddf5f

@bxyu-nvidia
bxyu-nvidia merged commit 03f0dfd into main Sep 6, 2026
38 checks passed
@bxyu-nvidia
bxyu-nvidia deleted the jveronvialard/arena-reasoning-only-scoring branch September 6, 2026 21:20
jkyi-nvidia pushed a commit that referenced this pull request Sep 11, 2026
…3097)

## What does this PR do?

score rollouts with policy responses marked as completed but containing
only reasoning content as zero, instead of treating them as failures and
excluding them from lmarena_v3 score calculation

## Checklist

- [X] I have read the [contributing
guidelines](https://docs.nvidia.com/nemo/gym/latest/contribute/development-setup).
- [X] The change is focused; unrelated "drive-by" edits are tracked as
separate issues/PRs.
- [X] Tests added or updated and pass locally, or N/A for docs-only /
non-code changes (so CI unit/server checks pass when applicable).
- [X] Pre-commit checks pass locally (`pre-commit run --all-files`) (so
CI lint/format/copyright pass).
- [X] All commits have DCO sign-off (`git commit -s`) (so the DCO check
passes).

Signed-off-by: Julien Veron Vialard <jveronvialar@nvidia.com>
Signed-off-by: Junkeun Yi <jkyi@nvidia.com>

This branch was previously deployed

1 inactive deployment
public β€” dfddf5f9 Deployed Sep 6, 2026 by copy-pr-bot[bot] via release / finalize / notify #2641
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants