Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace ONNX with BaseOnnx #24

Merged
merged 11 commits into from
Nov 9, 2020
Merged

Replace ONNX with BaseOnnx #24

merged 11 commits into from
Nov 9, 2020

Conversation

DrChainsaw
Copy link
Owner

Wip until BaseOnnx is registered

@DrChainsaw DrChainsaw changed the title Replace ONNX with BaseOnnx#onnxmutable Replace ONNX with BaseOnnx Oct 27, 2020
@codecov-io
Copy link

codecov-io commented Nov 9, 2020

Codecov Report

Merging #24 (78d1b9b) into master (f694977) will increase coverage by 1.51%.
The diff coverage is 92.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   91.67%   93.19%   +1.51%     
==========================================
  Files          12       14       +2     
  Lines         709      764      +55     
==========================================
+ Hits          650      712      +62     
+ Misses         59       52       -7     
Impacted Files Coverage Δ
src/ONNXmutable.jl 100.00% <ø> (ø)
src/deserialize/combine.jl 87.50% <82.60%> (ø)
src/baseonnx/write.jl 82.75% <82.75%> (ø)
src/deserialize/deserialize.jl 95.08% <88.23%> (+2.32%) ⬆️
src/baseonnx/read.jl 90.90% <90.90%> (ø)
src/deserialize/graphbuilder.jl 97.77% <97.56%> (+9.77%) ⬆️
src/baseonnx/BaseOnnx.jl 100.00% <100.00%> (ø)
src/deserialize/ops.jl 96.87% <100.00%> (ø)
src/serialize/serialize.jl 98.03% <100.00%> (+1.47%) ⬆️
src/validate.jl 100.00% <100.00%> (ø)
... and 4 more

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 f694977...78d1b9b. Read the comment docs.

@DrChainsaw DrChainsaw merged commit 9021a74 into master Nov 9, 2020
@DrChainsaw DrChainsaw deleted the baseonnx branch November 9, 2020 20:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants