Skip to content

[XPU] fix amp autocast typo #65525

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 1, 2024
Merged

[XPU] fix amp autocast typo #65525

merged 2 commits into from
Jul 1, 2024

Conversation

@dynamicheart
Copy link
Contributor

@dynamicheart dynamicheart commented Jun 27, 2024 •
edited
Loading

PR Category

Custom Device

PR Types

Bug fixes

Description

  • Fix amp autocast typo
  • The xpu_version from get_xpu_device_version() starts from 0(XPU1)
  • Now if the XPU device does not support AMP, it will print the following log:
/root/miniconda/lib/python3.8/site-packages/paddle/amp/auto_cast.py:541: UserWarning: XPUVersion.XPU2 does not support bfloat16 amp.

@paddle-bot
Copy link

paddle-bot bot commented Jun 27, 2024

你的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.

ZibinGuo

This comment was marked as outdated.

Sign in to view

ZibinGuo
ZibinGuo approved these changes Jul 1, 2024
View reviewed changes
Copy link
Contributor

@ZibinGuo ZibinGuo left a comment

Choose a reason for hiding this comment

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

LGTM

XiaociZhang
XiaociZhang reviewed Jul 1, 2024
View reviewed changes
Copy link
Contributor

@XiaociZhang XiaociZhang left a comment

Choose a reason for hiding this comment

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

LGTM

@QingshuChen QingshuChen merged commit 343a38e into PaddlePaddle:develop Jul 1, 2024
@houj04 houj04 added the XPU label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@QingshuChen QingshuChen QingshuChen approved these changes

+2 more reviewers

@XiaociZhang XiaociZhang XiaociZhang left review comments

@ZibinGuo ZibinGuo ZibinGuo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

XPU

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dynamicheart @QingshuChen @ZibinGuo @XiaociZhang @houj04