-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[xdoctest] reformat example code with google style in No.386-389 #58206
[xdoctest] reformat example code with google style in No.386-389 #58206
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
emmmm,注意审题,我们只需要确保 SKIP 里面有 reason 即可,而且 reason 并不是报错,这样的 reason 是一点意义都没有的,有些改动甚至影响了本来的语义(如 python/paddle/quantization/factory.py
)
third_party/mkldnn
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个要恢复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
嗯,才发现
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以看一下原来的 commit hash,然后 cd 到这个 submodule 里,checkout 回原来的 commit hash
58e7d57
to
2494696
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMeow 🐾
- out_vector(Tensor): A Tensor with shape [*, N, N] and data type of float32,float64, | ||
complex64 and complex128. The eigenvectors of eigh op. | ||
complex64 and complex128. The eigenvectors of eigh op. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
顺带 fix eigh 的 system message
@sunzhongkai588 来瞅瞅~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
预览好像还是不行..两个段落之间空行试一下?@enkilee
@megemini 佬,有空PR审核下? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…dlePaddle#58206) * fix * undo mkldnn * fix * fix sys msg * fix another sys msg --------- Co-authored-by: SigureMo <[email protected]>
PR types
Others
PR changes
Others
Description
386-389
flash_attention 里面2个函数
1、flash_attn_unpadded error: "
is_sm8x || is_sm75
check failed at /paddle/third_party/flashattn/csrc/flash_attn/flash_attn.cpp:293"2、scaled_dot_product_attention error: (InvalidArgument) flash_attn(): argument (position 5) must be double, but got NoneType (at ../paddle/fluid/pybind/op_function_common.cc:241)
在cpu和gpu下还是报错