-
Notifications
You must be signed in to change notification settings - Fork 740
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
add dummy interfaces for import timm #10395
Conversation
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.
LGTM
开 lazy mock 还需要这些空接口吗? |
有些是装饰器函数,比如 13 @parse_args("v", "v", "f")
---> 14 def symbolic_multi_label_nms(g, boxes, scores, iou_threshold):
15 boxes = opset11.unsqueeze(g, boxes, 0)
16 scores = opset11.unsqueeze(g, opset11.unsqueeze(g, scores, 0), 0) 有些是
有些是类型声明的,比如 |
CI failed when running job: Build cpu. PR label automerge has been removed |
开了 lazy mock 之后这些接口即使不存在应该也都不会报错? lazy mock 是 这里 的 lazy=True 参数 |
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. |
我试了下还真是 😅 用户当时使用的时候是没开 lazy,会直接报错 |
可以把 lazy 设成默认的,或者更新下文档推荐用户使用 lazy(现在文档里说 lazy 是为了调试方便,这个看法过时了) |
View latest API docs preview at: https://oneflow-staging.oss-cn-beijing.aliyuncs.com/docs/Oneflow-Inc/oneflow/pr/10395/ |
Speed stats:
|
#10395 这个 PR 里多此一举给 jit/\_\_init__.py 加了一个 warning,导致 import oneflow 的时候就会显示这个 warning,这里去掉
python3 -c "import timm" 输出如下,然后成功返回