RLOOTrainer & PPOv2Trainer - Modify Name for W&B Logged Table

Open Beginner friendly
#2,045 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
75/100
Issue type
Feature
Clarity
Clearly specified
Activity status
Quiet
Tech stack
python

Research direction

Start with the W&B table logging statements at trl/trainer/rloo_trainer.py#L511-L512 and trl/trainer/ppov2_trainer.py#L613-L614. Review how self.state.global_step is available at those entry points, then update the table naming so completions from different training steps remain separately inspectable in W&B. Done means both trainers produce step-specific table names.

Written by the indexing model from the issue text.

Description

✨ enhancement 🏋 PPO 🏋 RLOO 🙋 help from community wanted
Feature request

RLOOTrainer and PPOv2Trainer both log a table to W&B titled "completions". I would like the name to be changed to f"completions_step={self.state.global_step}" or something like this.

Motivation

Due to how W&B displays tables in the WebUI, logging the table under the same name "completions" means that one can only see the most recent table of completions, rather than tables of completions at different points during learning.

I would like to be able to manually inspect how completions are changing during learning.

I can probably pull this information out of the W&B API, but this is too much effort. I want to be able to manually inspect easily.

Your contribution

I would be happy to submit a PR for this.

Dominant language
Python
Stars
19.3k
Forks
3k
Avg merge
1d 9h
Merged PRs (30d)
182

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from huggingface/trl

All issues in huggingface/trl

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.