Skip to content

Update ppo #10912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Aug 7, 2025
Merged

Update ppo #10912

merged 19 commits into from
Aug 7, 2025

Conversation

dddd-d
Copy link
Contributor

@dddd-d dddd-d commented Aug 6, 2025

Before submitting

  • Lint code. If there are lint issues, please format the code first.
# Install and register `pre-commit` in the project folder
pip install pre-commit && pre-commit install

# Process previous code files separately
pre-commit run --file XXXX.py
  • Add test cases into tests folder. If there are codecov issues, please add tests cases first.

PR types

New features

PR changes

Models

Description

update ppo

原PR地址(含PR view):#10884

Copy link

paddle-bot bot commented Aug 6, 2025

Thanks for your contribution!

Copy link
Contributor

@DesmonDay DesmonDay left a comment

Choose a reason for hiding this comment

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

LGTM (备注:在原来的PR已经整体review过,新提PR主要是为了触发CI)

@DesmonDay DesmonDay merged commit 3e9d351 into PaddlePaddle:develop Aug 7, 2025
11 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants