-
Notifications
You must be signed in to change notification settings - Fork 857
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
Reduce arg combinations of sgd/adam/adamw unittest to prevent timeout #9886
Conversation
Code got formatted by CI. Please request CI again if you still want to have this PR merged. If the PR is from a forked repo, please download the patch files from the GitHub Actions web page and apply them locally. |
Speed stats:
|
arg_dict["tensor_num"] = [1, 4] | ||
for arg in GenArgDict(arg_dict): | ||
compare_with_numpy_sgd(test_case, **arg) | ||
|
||
@autotest(n=10) |
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.
这个是不是也不用加,遍历 arg_dict 本身就循环快 300 次了
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.
这个是不是也不用加,遍历 arg_dict 本身就循环快 300 次了
这个我也比较纠结;这里加上是我考虑因为参数里面本身就有随机量,多跑几次能多取一些值,覆盖得稍微全面一些
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.
感觉 300 次足够了,很多参数是正交的,不必把每种组合都跑到
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.
感觉 300 次足够了,很多参数是正交的,不必把每种组合都跑到
ok,那我这里去掉
Speed stats:
|
CI failed when running job: cuda-module. PR label automerge has been removed |
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9886/ |
…om/Oneflow-Inc/oneflow into fix_reduce_arglist_of_sgd_unittest
Code got formatted by CI. Please request CI again if you still want to have this PR merged. If the PR is from a forked repo, please download the patch files from the GitHub Actions web page and apply them locally. |
…om/Oneflow-Inc/oneflow into fix_reduce_arglist_of_sgd_unittest
Speed stats:
|
Code got formatted by CI. Please request CI again if you still want to have this PR merged. If the PR is from a forked repo, please download the patch files from the GitHub Actions web page and apply them locally. |
Speed stats:
|
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9886/ |
Speed stats:
|
Speed stats:
|
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9886/ |
Speed stats:
|
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9886/ |
1 similar comment
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9886/ |
sgd / adam / adamw 的单测中参数组合爆炸导致 ci 容易超时,这里对部分参数进行了采样来减少组合数。
修改之后,slowest top 50 已经没有这几个单测了。