Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Convert tf.add_n using keras _builtin #406

Merged
merged 16 commits into from
Mar 9, 2020
Merged

Convert tf.add_n using keras _builtin #406

merged 16 commits into from
Mar 9, 2020

Conversation

jiafatom
Copy link
Collaborator

@jiafatom jiafatom commented Mar 9, 2020

No description provided.

jiafatom and others added 12 commits February 26, 2020 11:38
@jiafatom
Add vgg16 and nasnet to v2 application
d674930
@jiafatom
Add vgg16 and nasnet to v2 application
c731d6f
@jiafatom
Merge branch 'test_tfl' of https://github.com/jiafatom/keras-onnx int…
e389ae3 
…o test_tfl
@jiafatom
Merge branch 'master' into test_tfl
d200756
@wenbingl @jiafatom
Add DepthwiseConv2d to subclassed model and efficient-net test cases (o…
4dd8869 
…nnx#394)

* Efficient-net test cases.

* Add DepthwiseConv2d to subclassed model.

* disable efficient net test cases.
@jiafatom
Merge branch 'test_tfl' of https://github.com/jiafatom/keras-onnx
7f9add8
@jiafatom
Merge branch 'master' of https://github.com/jiafatom/keras-onnx
1bb0e71
@jiafatom
Merge branch 'master' of https://github.com/onnx/keras-onnx
0cbfd4b
@jiafatom
Merge branch 'master' of https://github.com/onnx/keras-onnx
59cf5dd
@jiafatom
Merge branch 'master' of https://github.com/onnx/keras-onnx
99c9428
@jiafatom
Merge branch 'master' of https://github.com/jiafatom/keras-onnx
86559d9
@jiafatom
Convert tf.add_n using keras _builtin
b61b05c
@jiafatom jiafatom requested review from wenbingl and vinitra-zz March 9, 2020 21:15
vinitra-zz
vinitra-zz approved these changes Mar 9, 2020
View reviewed changes
Copy link
Member

@vinitra-zz vinitra-zz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM after CIs pass, made small suggestion.

tests/test_layers.py Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Mar 9, 2020

This pull request introduces 1 alert when merging b61b05c into 992822e - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@lgtm-com
Copy link

lgtm-com bot commented Mar 9, 2020

This pull request introduces 1 alert when merging 655b22e into 992822e - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@jiafatom jiafatom merged commit eed910b into onnx:master Mar 9, 2020
@jiafatom jiafatom deleted the add_n branch March 9, 2020 21:46
@jiafatom jiafatom mentioned this pull request Mar 9, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@vinitra-zz vinitra-zz vinitra-zz approved these changes

@wenbingl wenbingl wenbingl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jiafatom @wenbingl @vinitra-zz