#592 celu selu gelu

Merged
zoulq merged 10 commits from lzh/MSAdapter:opt_0703 into master 10 months ago
lzh commented 10 months ago
lzh changed title from celu selu gelu to [WIP]celu selu gelu 10 months ago
lzh changed title from [WIP]celu selu gelu to celu selu gelu 10 months ago
frelam reviewed 10 months ago
@@ -885,2 +884,3 @@
| nn.CELU | Partly supported | inplace not support on GRAPH mode |
| nn.SELU | Partly supported | [Function is constrained](ConstraintList_en.md) |
| nn.CELU | Partly supported | [Function is constrained](ConstraintList_en.md) |
| nn.GELU | Supported | |
frelam commented 10 months ago
nn.GELU当前缺少`approximate`参数, 而且当前调用的是ms.ops.GELU, 该算子只是满足计算approximiate为tanh的场景。 需要改为调用ms.ops.gelu
lzh commented 10 months ago
大写的GELU SELU CELU不在这个pr的整改范围,这里误改了,现在先改回去了。
frelam reviewed 10 months ago
ConstraintList.md
@@ -250,1 +250,4 @@
| functional.mish | 1.`inplace`不支持图模式;2.不支持float64 |
| functional.selu | `inplace`不支持图模式 |
| functional.celu | 1.`inplace`不支持图模式;2.不支持float64 |
| functional.grid_sample | 1.图模式下不支持等号赋值关键字参数。2.不支持`mode='bicubic'` |
frelam commented 10 months ago
"1.图模式下不支持等号赋值关键字参数。" 有没有更好的表达方式。
frelam commented 10 months ago
是不是所有接口图模式下都有这个问题, 不能用key-value的方式。
lzh commented 10 months ago
nn类接口在图模式下都有这个问题,暂时不体现在用户侧约束了,已删除
frelam reviewed 10 months ago
testing/ut/pytorch/nn/test_init.py
@@ -10,3 +10,3 @@

def test_dirac_():
t_w1 = torch.empty(3, 16, 5, 5)
t_w1 = torch.empty(3, 16, 5, 5, dtype=torch.float64)
frelam commented 10 months ago
这里是为了测试float64的支持度?
lzh commented 10 months ago
对,已验证,通过
frelam reviewed 10 months ago
frelam reviewed 10 months ago
@@ -26,3 +25,3 @@

def test_pdist2():
data = np.arange(0, 12).reshape(4, 3).astype(np.float32)
data = np.arange(0, 12).reshape(4, 3).astype(np.float64)
frelam commented 10 months ago
ascend上可以支持float64吗
lzh commented 10 months ago
已验证,支持
zoulq merged commit 112dae0f60 into master 10 months ago
The pull request has been merged as 112dae0f60.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.