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
Refine module test using auto test by yaochi #5484
Conversation
…nc/oneflow into refine_autotest_yaochi
@@ -20,6 +20,7 @@ | |||
|
|||
import oneflow.experimental as flow |
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.
我感觉这些数学有关的操作应该统一放到test_math_ops.py里面,没必要单独开文件了,迟哥觉得呢
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.
是的,按道理应该不要分成多个文件。
现在这样做,是因为之前多人分工,由于大家的编程习惯难同意,同时编辑一个文件,会反复造成 conflict。为了降低沟通成本,就让大家各自开文件。
最近 python 算子层的工作还没告一段落(还需要改 inplace)。等这个弄完,我再统一过一遍汇总吧,你看可以么。
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.
好的,同意
完成 #5352 分配到我的任务。
var 的测试过程中,发现后向有问题,这个PR中就不重构, 后续单独提交PR,修复 var 的后向问题,并且重构 var 的测试。