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

upgrade to 1.7 #384

Merged
merged 3 commits into from
Apr 9, 2020
Merged

upgrade to 1.7 #384

merged 3 commits into from
Apr 9, 2020

Conversation

wenbingl
Copy link
Member

@wenbingl wenbingl commented Apr 9, 2020

No description provided.

@wenbingl wenbingl requested review from jiafatom and xadupre April 9, 2020 18:04
@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2020

This pull request fixes 1 alert when merging 5f803a9 into 6546438 - view on LGTM.com

fixed alerts:

  • 1 for Except block handles 'BaseException'

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2020

This pull request fixes 1 alert when merging 2f18905 into 6546438 - view on LGTM.com

fixed alerts:

  • 1 for Except block handles 'BaseException'

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2020

This pull request introduces 2 alerts and fixes 5 when merging 5657e36 into 6546438 - view on LGTM.com

new alerts:

  • 2 for Use of the return value of a procedure

fixed alerts:

  • 3 for 'import *' may pollute namespace
  • 1 for Except block handles 'BaseException'
  • 1 for Unused import

@wenbingl wenbingl merged commit 78b12fd into onnx:master Apr 9, 2020
@wenbingl wenbingl deleted the o17 branch April 9, 2020 23:33
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