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

GNNE-1714:Fix/quant by json #1106

Merged
merged 19 commits into from
Oct 12, 2023
Merged

GNNE-1714:Fix/quant by json #1106

merged 19 commits into from
Oct 12, 2023

Conversation

uranus0515
Copy link
Contributor

Add the operation of assigning OP's name to some transforms so that the json file used to guide quantification can be correctly exported.

@uranus0515 uranus0515 changed the title Fix/quant by json GNNE-1714:Fix/quant by json Oct 12, 2023
@github-actions
Copy link

github-actions bot commented Oct 12, 2023

Test Results

5 591 tests  ±0   5 591 ✔️ ±0   10m 31s ⏱️ - 1m 18s
       6 suites ±0          0 💤 ±0 
       6 files   ±0          0 ±0 

Results for commit 9ad57bb. ± Comparison against base commit c95082d.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (c95082d) 79.22% compared to head (9ad57bb) 79.23%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1106   +/-   ##
=======================================
  Coverage   79.22%   79.23%           
=======================================
  Files         689      689           
  Lines       24267    24322   +55     
=======================================
+ Hits        19226    19271   +45     
- Misses       5041     5051   +10     
Flag Coverage Δ
integration 41.12% <56.33%> (+0.03%) ⬆️
unit 72.17% <47.88%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/Nncase.Importer/Onnx/Conv2D.cs 78.84% <100.00%> (+1.76%) ⬆️
src/Nncase.Importer/Onnx/Gemm.cs 90.00% <100.00%> (+1.11%) ⬆️
src/Nncase.Importer/Onnx/MatMul.cs 100.00% <100.00%> (ø)
src/Nncase.Importer/TFLite/Activations.cs 72.41% <100.00%> (+3.18%) ⬆️
src/Nncase.Importer/TFLite/Binary.cs 66.66% <100.00%> (+23.80%) ⬆️
src/Nncase.Importer/TFLite/Conv2DTranspose.cs 100.00% <100.00%> (ø)
src/Nncase.Importer/TFLite/MatMul.cs 74.46% <100.00%> (+3.73%) ⬆️
...Nncase.Passes/Rules/Neutral/AddRangeOfAndMarker.cs 98.95% <100.00%> (+0.05%) ⬆️
...c/Nncase.Passes/Rules/Neutral/BatchNormToBinary.cs 100.00% <100.00%> (ø)
src/Nncase.Passes/Rules/Neutral/ReshapeMatMul.cs 96.96% <100.00%> (ø)
... and 4 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@uranus0515 uranus0515 merged commit 01da2d0 into master Oct 12, 2023
46 checks passed
@uranus0515 uranus0515 deleted the fix/quant_by_json branch October 12, 2023 05:28
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.

None yet

2 participants