DrChainsaw / ONNXNaiveNASflux.jl

Import/export ONNX models
MIT License
43 stars 2 forks source link

Add InstanceNorm #79

Closed itan1 closed 1 year ago

itan1 commented 1 year ago

I added InstanceNorm in a similar way to how BatchNorm was added.

It will only work for Flux.InstanceNorm layers with affine=true and track_stats=false unfortunately, since the ONNX InstanceNormalization layer requires that (for BatchNorm we also require affine=true though).

codecov[bot] commented 1 year ago

Codecov Report

Merging #79 (9b5db38) into master (ee7f5f0) will increase coverage by 0.00%. The diff coverage is 100.00%.

:exclamation: Current head 9b5db38 differs from pull request most recent head 8902277. Consider uploading reports for the commit 8902277 to get more accurate results

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   95.33%   95.33%           
=======================================
  Files          13       13           
  Lines        1007     1008    +1     
=======================================
+ Hits          960      961    +1     
  Misses         47       47           
Impacted Files Coverage Δ
src/deserialize/ops.jl 96.92% <100.00%> (+0.01%) :arrow_up:
src/serialize/serialize.jl 95.85% <100.00%> (+0.17%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more