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

Increase SequenceMap version to 17 #4133

Merged
merged 4 commits into from Apr 15, 2022

Conversation

jantonguirao
Copy link
Contributor

Signed-off-by: Joaquin Anton janton@nvidia.com

Description

  • Increasing SequenceMap version to 17, since it was merged after the release of version 16.

Motivation and Context

  • Since Opset 16 was released before SequenceMap was merged, this new operator should belong to the new opset.

@jantonguirao jantonguirao requested a review from a team as a code owner April 13, 2022 16:41
Signed-off-by: Joaquin Anton <janton@nvidia.com>
@garymm garymm added this to the 1.12 milestone Apr 13, 2022
Copy link
Member

@jcwchen jcwchen left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!

onnx/defs/operator_sets.h Show resolved Hide resolved
@jcwchen jcwchen enabled auto-merge (squash) April 15, 2022 23:07
@jcwchen jcwchen merged commit fb1f721 into onnx:main Apr 15, 2022
@jcwchen jcwchen added the preprocessing data preprocessing label Apr 21, 2022
broune pushed a commit to broune/onnx that referenced this pull request May 6, 2023
* Increase SequenceMap version to 17

Signed-off-by: Joaquin Anton <janton@nvidia.com>

* Update comment about fwd declarations for opset 17

Signed-off-by: Joaquin Anton <janton@nvidia.com>

Co-authored-by: Chun-Wei Chen <jacky82226@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preprocessing data preprocessing
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants