Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@p-wysocki
Copy link
Contributor

Description

Mark the VERSION_NUMBER as the final 1.13.1 release. Please do not merge until agreed that we're ready to push to PyPI.

@p-wysocki p-wysocki requested a review from a team as a code owner February 9, 2023 15:56
Signed-off-by: p-wysocki <[email protected]>
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.

Please fix the lint error here:

===> check flake8...
  onnx/reference/ops/op_string_normalizer.py:89:17: B028 No explicit stacklevel keyword argument found. The warn method from the warnings module uses a stacklevel of 1 by default. This will only show a stack trace for the line on which the warn method is called. It is therefore recommended to use a stacklevel of 2 or greater to provide more information to the user.

It was not caught previously, because the lint CI has been updated recently. Thanks!

Signed-off-by: p-wysocki <[email protected]>
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.

I think we can finalize this release since there isn't any regression.

@jcwchen jcwchen merged commit ad834eb into onnx:rel-1.13.1 Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants