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.

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

fix cusparse compile problem, test=develop #36199

Merged
merged 3 commits into from
Sep 29, 2021
Merged

fix cusparse compile problem, test=develop #36199

merged 3 commits into from
Sep 29, 2021

Conversation

Liu-xiandong
Copy link
Member

PR types

Bug fixes

PR changes

Others

Describe

本PR解决两个问题：

  1. 对test_sparse_attention_op.py中的进行修改，变量命名有误；
  2. 限定cusparse的平台，目前发现在windows和ARM中不支持相关API接口。

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

xingfeng01
xingfeng01 previously approved these changes Sep 29, 2021
View reviewed changes
@tianshuo78520a
Copy link
Contributor

LGTM

AnnaTrainingG
AnnaTrainingG previously approved these changes Sep 29, 2021
View reviewed changes
zkh2016
zkh2016 previously approved these changes Sep 29, 2021
View reviewed changes
zhwesky2010
zhwesky2010 reviewed Sep 29, 2021
View reviewed changes
Copy link
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

改动了文件权限，10644->10755，需要在本地再操作一下

@Liu-xiandong
Copy link
Member Author

改动了文件权限，10644->10755，需要在本地再操作一下

Done.

@zhwesky2010 zhwesky2010 merged commit 3eb5071 into PaddlePaddle:develop Sep 29, 2021
Liu-xiandong added a commit to Liu-xiandong/Paddle that referenced this pull request Oct 14, 2021
@Liu-xiandong
fix cusparse compile problem, test=develop (PaddlePaddle#36199)
86389fb 
* fix cusparse compile problem, test=develop

* Modify file permissions
@Liu-xiandong Liu-xiandong mentioned this pull request Oct 14, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zkh2016 zkh2016 zkh2016 approved these changes

@AnnaTrainingG AnnaTrainingG AnnaTrainingG approved these changes

@zhwesky2010 zhwesky2010 zhwesky2010 approved these changes

@xingfeng01 xingfeng01 xingfeng01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Liu-xiandong @tianshuo78520a @zkh2016 @AnnaTrainingG @zhwesky2010 @xingfeng01