chainer / onnx-chainer

Add-on package for ONNX format support in Chainer
MIT License
85 stars 24 forks source link

fix to manage each var is renamed or not #155

Closed disktnk closed 5 years ago

disktnk commented 5 years ago

fixes #154

before

image

export with output_names=['out_tanh', 'out_sigmoid'], outputs are renamed but interim node output are not renamed, such as v15

after

image

codecov-io commented 5 years ago

Codecov Report

Merging #155 into master will increase coverage by <.01%. The diff coverage is 95.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
+ Coverage   85.29%   85.29%   +<.01%     
==========================================
  Files          22       22              
  Lines        1122     1129       +7     
==========================================
+ Hits          957      963       +6     
- Misses        165      166       +1
Impacted Files Coverage Δ
onnx_chainer/export.py 92.54% <100%> (+0.02%) :arrow_up:
onnx_chainer/context.py 96.29% <90%> (-3.71%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 18d9330...ceb31d4. Read the comment docs.

disktnk commented 5 years ago

ChainerCI, test this please

pfn-ci-bot commented 5 years ago

Successfully created a job for commit 2c5dd76 (6f15d9d):

disktnk commented 5 years ago

ChainerCI, test this please

pfn-ci-bot commented 5 years ago

Successfully created a job for commit ceb31d4 (e5cf8e9):