#805 fix some bug

Merged
zoulq merged 1 commits from erpim_1205 into master 1 year ago
Erpim commented 1 year ago
zoulq reviewed 1 year ago
mindtorch/torch/functional.py
@@ -1692,3 +1690,3 @@
return _out_inplace_assign_with_adapter_tensor(out, output, "roll")

def rot90(input, k, dims, *, out=None):
def rot90(input, k=1, dims=[0, 1], *, out=None): # pylint: disable=W0102
zoulq commented 1 year ago
这个pylint屏蔽的原因是什么?
Erpim commented 1 year ago
pylint检查默认属性是list是比较危险的行为,因为默认值只会初始化一次,list是个变量,后续修改可能在原始list上直接修改.目前这个场景下没有识别出问题,且torch默认值就是list
zoulq reviewed 1 year ago
@@ -0,0 +11,4 @@

def test_mul1():
torch_out = torch.mul(torch.tensor(0.5), 0.5)
ms_out = ms_torch.mul(ms_torch.tensor(0.5), 0.5)
zoulq commented 1 year ago
写成图模式下进图的方式
Erpim commented 1 year ago
done
zoulq reviewed 1 year ago
testing/ut/pytorch/tensor/test_tensor2.py
@@ -688,0 +697,4 @@

def test_contiguous():
ms_tensor = ms_torch.Tensor(2, 3)
output = ms_tensor.contiguous()
zoulq commented 1 year ago
同上,图模式下要真正进图
Erpim commented 1 year ago
done
Erpim changed title from fix some bug to [WIP]fix some bug 1 year ago
Erpim changed title from [WIP]fix some bug to fix some bug 1 year ago
zoulq reviewed 1 year ago
rank_0/om/analyze_fail.ir
@@ -0,0 +1,306 @@
# ===============================================================================================
zoulq commented 1 year ago
删掉文件
zoulq merged commit 015ac2871b into master 1 year ago
The pull request has been merged as 015ac2871b.
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.