Skip to content
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

deal with multi input tensors #99

Merged
merged 17 commits into from
Oct 18, 2022
Merged

deal with multi input tensors #99

merged 17 commits into from
Oct 18, 2022

Conversation

BBuf
Copy link
Contributor

@BBuf BBuf commented Oct 11, 2022

#close #98

libai t5:

图片

@BBuf BBuf marked this pull request as ready for review October 12, 2022 07:40
@BBuf BBuf requested review from oneflow-ci-bot and CPFLAME and removed request for oneflow-ci-bot October 12, 2022 07:40
@BBuf BBuf requested review from oneflow-ci-bot and removed request for oneflow-ci-bot October 13, 2022 01:52
@CPFLAME
Copy link
Contributor

CPFLAME commented Oct 18, 2022

验证过了, 功能没有问题了.
image

然后保存好的model也可以用大老师的onnx_simplifier. 先点approve了. 可以解答一下上面的comments疑问

@BBuf BBuf requested review from oneflow-ci-bot and removed request for oneflow-ci-bot October 18, 2022 06:15
@BBuf
Copy link
Contributor Author

BBuf commented Oct 18, 2022

验证过了, 功能没有问题了. image

然后保存好的model也可以用大老师的onnx_simplifier. 先点approve了. 可以解答一下上面的comments疑问

simplify也加上了

@BBuf BBuf removed the request for review from oneflow-ci-bot October 18, 2022 06:19
@BBuf BBuf requested review from oneflow-ci-bot and removed request for oneflow-ci-bot October 18, 2022 07:20
@BBuf BBuf requested review from oneflow-ci-bot and removed request for oneflow-ci-bot October 18, 2022 07:40
@BBuf BBuf requested review from oneflow-ci-bot and removed request for oneflow-ci-bot October 18, 2022 07:44
@BBuf BBuf merged commit e2a68d7 into main Oct 18, 2022
@BBuf BBuf deleted the fix_t5_export_onnx_bug branch October 18, 2022 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

调用convert_to_onnx_and_check接口报错 "Error No Op registered for Range with domain_version of 10"
2 participants