Skip to content
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

[XPU] change cuda_rng_state to rng_state in fleet random #54077

Merged
merged 1 commit into from
May 24, 2023

Conversation

houj04
Copy link
Contributor

@houj04 houj04 commented May 24, 2023

PR types

New features

PR changes

Others

Description

fleet中的RNGStatesTracker类,在记录和设置状态的时候,使用的函数是get_cuda_rng_stateset_cuda_rng_state。这样的话在XPU下面就会表现异常,例如各个rank拿不到自己的专属种子,等等。

因此本PR将其改成了调用get_rng_stateset_rng_state,这样在GPU和XPU下面都能用。

@paddle-bot
Copy link

paddle-bot bot commented May 24, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels May 24, 2023
@paddle-bot
Copy link

paddle-bot bot commented May 24, 2023

❌ The PR is not created using PR's template. You can refer to this Demo.
Please use PR's template, it helps save our maintainers' time so that more developers get helped.

@wuhuachaocoding
Copy link
Contributor

LGTM

Copy link
Contributor

@wuhuachaocoding wuhuachaocoding left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@sljlp sljlp left a comment

Choose a reason for hiding this comment

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

LGTM

@sljlp sljlp merged commit 23baa8c into PaddlePaddle:develop May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants