Skip to content

Conversation

sharvin2187
Copy link
Contributor

Description

Checklist

  • I have read the contribution guidelines.
  • I have coordinated with the docs team (mldocs@microsoft.com) if this PR deletes files or changes any file names or file extensions.
  • Pull request includes test coverage for the included changes.
  • This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

@sharvin2187 sharvin2187 requested a review from rtanase as a code owner May 28, 2025 09:31
@sharvin2187 sharvin2187 force-pushed the shjondhale/grpo_followups branch from 67d6e48 to 54621c3 Compare May 28, 2025 09:32
@sharvin2187 sharvin2187 requested a review from babu-namburi as a code owner May 29, 2025 06:03
@sharvin2187 sharvin2187 force-pushed the shjondhale/grpo_followups branch from c81544c to 17d3334 Compare May 29, 2025 06:05
sharvin2187 and others added 2 commits May 29, 2025 13:11
Co-authored-by: Gayatri Penumetsa <181455625+gpenumetsa-msft@users.noreply.github.com>
Co-authored-by: Gayatri Penumetsa <181455625+gpenumetsa-msft@users.noreply.github.com>
Copy link
Contributor

@gpenumetsa-msft gpenumetsa-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@sharvin2187 sharvin2187 merged commit 7682785 into main May 29, 2025
7 checks passed
@sharvin2187 sharvin2187 deleted the shjondhale/grpo_followups branch May 29, 2025 08:20
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.

4 participants