Skip to content

Add sequence identity for opset 14 & fix sequence insert - #7335

Merged
BowenBao merged 2 commits into
masterfrom
bowbao/sequence_insert_fix
Apr 28, 2021
Merged

Add sequence identity for opset 14 & fix sequence insert#7335
BowenBao merged 2 commits into
masterfrom
bowbao/sequence_insert_fix

Conversation

@BowenBao

@BowenBao BowenBao commented Apr 13, 2021

Copy link
Copy Markdown
Contributor

Needs #7177 to merge first. Merged.

Description:

  • Fix SequenceInsert with last position, which is equal to the current sequence length.
  • Implement Identity to support sequence input for opset 14.

Motivation and Context

  • Required to export Huggingface/transformers T5 with beam search.

@BowenBao
BowenBao requested a review from a team as a code owner April 13, 2021 23:02
@BowenBao BowenBao changed the title Add sequence identity for opset 14 & fix sequence insert [WIP] Add sequence identity for opset 14 & fix sequence insert Apr 14, 2021
@BowenBao
BowenBao force-pushed the bowbao/sequence_insert_fix branch from 9f75f9d to 8124bfd Compare April 21, 2021 22:31
cleanup changes, add testcases, and move identity to opset 14

fix identity opset 13 registration

add hash for new kernel

fix test case
@BowenBao
BowenBao force-pushed the bowbao/sequence_insert_fix branch from 800132b to 97bc10b Compare April 27, 2021 18:47
@BowenBao BowenBao changed the title [WIP] Add sequence identity for opset 14 & fix sequence insert Add sequence identity for opset 14 & fix sequence insert Apr 27, 2021
@BowenBao

Copy link
Copy Markdown
Contributor Author

cc @pranavsharma for review.

@BowenBao
BowenBao requested a review from pranavsharma April 27, 2021 21:04
Comment thread onnxruntime/core/providers/cpu/sequence/sequence_ops.cc
@BowenBao
BowenBao merged commit c584d48 into master Apr 28, 2021
@BowenBao
BowenBao deleted the bowbao/sequence_insert_fix branch April 28, 2021 20:26
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.

2 participants