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

support lstm opr in onnx2mge convert #64

Merged
merged 1 commit into from
Mar 3, 2022
Merged

Conversation

a419775258
Copy link
Collaborator

add support for lstm in onnx2mge converter

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


王博文 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@dingshaohua960303
Copy link
Collaborator

可以补个测试, others LGTM

Copy link
Collaborator

@daisycx daisycx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,可以更新下算子列表

@a419775258 a419775258 force-pushed the add-lstm-support branch 3 times, most recently from cdce730 to 74ca235 Compare February 17, 2022 05:47
@a419775258 a419775258 force-pushed the add-lstm-support branch 4 times, most recently from a17a173 to 7ffde40 Compare March 1, 2022 05:12
@a419775258
Copy link
Collaborator Author

@dingshaohua960303 麻烦看一下,有个代码检查的 ci 没跑过,原因是使用了一个 新版本 megengine 才有的 module。如果不影响的话,能否你动手合入一下?

@a419775258 a419775258 merged commit 5a9d581 into master Mar 3, 2022
@a419775258 a419775258 deleted the add-lstm-support branch March 3, 2022 08:37
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.

4 participants