GNNE-1876 Feature/online compile
- add online cost evaluator provider
- add auto merge with online cost evaluator test
Codecov Report
Patch coverage: 80.10% and project coverage change: +0.38 :tada:
Comparison is base (
19e9859) 77.99% compared to head (eb31291) 78.37%.
Additional details and impacted files
@@ Coverage Diff @@
## master #1008 +/- ##
==========================================
+ Coverage 77.99% 78.37% +0.38%
==========================================
Files 634 634
Lines 21664 21750 +86
==========================================
+ Hits 16896 17047 +151
+ Misses 4768 4703 -65
| Flag | Coverage Δ | |
|---|---|---|
| integration | 44.51% <41.36%> (+0.08%) |
:arrow_up: |
| unit | 70.04% <80.10%> (+0.52%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Impacted Files | Coverage Δ | |
|---|---|---|
| src/Nncase.Core/CoreApplicationPart.cs | 100.00% <ø> (ø) |
|
| ...rc/Nncase.EGraph/Passes/EGraphExtractExtensions.cs | 62.50% <ø> (-5.69%) |
:arrow_down: |
| src/Nncase.Evaluator/Buffers/Uninitialized.cs | 50.00% <ø> (ø) |
|
| src/Nncase.Evaluator/CostEvaluateContext.cs | 52.94% <0.00%> (+52.94%) |
:arrow_up: |
| src/Nncase.Evaluator/CostEvaluateVisitor.cs | 65.00% <0.00%> (+65.00%) |
:arrow_up: |
| src/Nncase.Evaluator/ICostEvaluator.cs | 100.00% <ø> (ø) |
|
| ...rc/Nncase.Evaluator/OnlineCostEvaluatorProvider.cs | 74.32% <74.32%> (ø) |
|
| src/Nncase.Core/CompileSession.cs | 96.15% <75.00%> (-3.85%) |
:arrow_down: |
| src/Nncase.Evaluator/CostEvaluateProvider.cs | 56.25% <75.00%> (+6.25%) |
:arrow_up: |
| src/Nncase.EGraph/CostModel/EGraphCostEvaluator.cs | 83.01% <80.95%> (-5.70%) |
:arrow_down: |
| ... and 9 more |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Test Results
5 472 tests ±0 5 472 :heavy_check_mark: ±0 19m 18s :stopwatch: - 5m 31s 5 suites ±0 0 :zzz: ±0 5 files ±0 0 :x: ±0
Results for commit eb31291d. ± Comparison against base commit 19e98598.
:recycle: This comment has been updated with latest results.