Skip to content
This repository was archived by the owner on Jan 15, 2025. It is now read-only.

Comments

refactor Confusion Matrix API#904

Merged
hcyang merged 2 commits intomasterfrom
hunyang/dispatcher_refactorConfusionMatrixAPI
Jul 18, 2020
Merged

refactor Confusion Matrix API#904
hcyang merged 2 commits intomasterfrom
hunyang/dispatcher_refactorConfusionMatrixAPI

Conversation

@hcyang
Copy link
Contributor

@hcyang hcyang commented Jul 18, 2020

Refactor Confusion Matrix for bf-cli-orchestrator plugins.

@hcyang hcyang requested review from daveta and tsuwandy July 18, 2020 09:52
@hcyang hcyang requested a review from munozemilio as a code owner July 18, 2020 09:52
@codecov-commenter
Copy link

Codecov Report

Merging #904 into master will decrease coverage by 1.86%.
The diff coverage is 31.99%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #904      +/-   ##
==========================================
- Coverage   58.00%   56.13%   -1.87%     
==========================================
  Files         217      219       +2     
  Lines       15079    15615     +536     
  Branches     2130     2228      +98     
==========================================
+ Hits         8746     8765      +19     
- Misses       5766     6287     +521     
+ Partials      567      563       -4     
Impacted Files Coverage Δ
packages/dispatcher/src/data/AppColumnarData.ts 93.54% <ø> (-6.46%) ⬇️
...ispatcher/src/data/AppEntityAnnotatedCorpusData.ts 92.00% <ø> (-8.00%) ⬇️
packages/dispatcher/src/data/AppLuData.ts 87.09% <ø> (-6.24%) ⬇️
packages/dispatcher/src/data/data.ts 80.67% <ø> (ø)
...athematics/mathematics_helper/MathematicsHelper.ts 37.28% <ø> (-6.80%) ⬇️
...n/abstract_base_evaluator/AbstractBaseEvaluator.ts 27.27% <ø> (ø)
...l/evaluation/cross_validation/AppCrossValidator.ts 77.62% <ø> (ø)
.../model/evaluation/report/AppDataProfileReporter.ts 0.00% <ø> (ø)
...aluation/report/AppModelMetaDataProfileReporter.ts 0.00% <ø> (ø)
...rc/model/evaluation/report/AppThresholdReporter.ts 0.00% <0.00%> (ø)
... and 83 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 6bd5cf5...f93efe8. Read the comment docs.

@hcyang hcyang merged commit 9de7f4c into master Jul 18, 2020
@munozemilio munozemilio deleted the hunyang/dispatcher_refactorConfusionMatrixAPI branch February 12, 2021 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants