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

Comments

Refactor confusion matrix api .1#912

Merged
hcyang merged 2 commits intomasterfrom
hunyang/dispatcher_refactorConfusionMatrixAPI1
Jul 27, 2020
Merged

Refactor confusion matrix api .1#912
hcyang merged 2 commits intomasterfrom
hunyang/dispatcher_refactorConfusionMatrixAPI1

Conversation

@hcyang
Copy link
Contributor

@hcyang hcyang commented Jul 27, 2020

No description provided.

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

codecov-commenter commented Jul 27, 2020

Codecov Report

Merging #912 into master will decrease coverage by 2.39%.
The diff coverage is 15.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #912      +/-   ##
==========================================
- Coverage   58.18%   55.78%   -2.40%     
==========================================
  Files         220      219       -1     
  Lines       16978    16161     -817     
  Branches     2350     2319      -31     
==========================================
- Hits         9878     9016     -862     
- Misses       6508     6571      +63     
+ Partials      592      574      -18     
Impacted Files Coverage Δ
...odel/evaluation/cross_validation/CrossValidator.ts 87.67% <ø> (ø)
packages/dispatcher/src/utility/utility.ts 42.80% <1.45%> (-9.07%) ⬇️
packages/dispatcher/src/utility/AppUtility.ts 71.87% <11.11%> (-23.78%) ⬇️
...patcher/src/data_structure/DictionaryMapUtility.ts 12.58% <16.98%> (+0.38%) ⬆️
...athematics/confusion_matrix/ConfusionMatrixBase.ts 91.20% <92.10%> (-0.11%) ⬇️
...rc/mathematics/confusion_matrix/ConfusionMatrix.ts 100.00% <100.00%> (ø)
...mathematics/confusion_matrix/AppConfusionMatrix.ts 6.66% <0.00%> (-18.92%) ⬇️
packages/command/src/readpipeddata.ts 65.21% <0.00%> (-18.12%) ⬇️
packages/qnamaker/src/commands/qnamaker/index.ts 66.66% <0.00%> (-16.67%) ⬇️
...kages/luis/src/commands/luis/application/create.ts 60.00% <0.00%> (-13.34%) ⬇️
... and 119 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 4c45f4a...193269a. Read the comment docs.

@hcyang hcyang merged commit f10d79c into master Jul 27, 2020
@munozemilio munozemilio deleted the hunyang/dispatcher_refactorConfusionMatrixAPI1 branch July 27, 2020 13:42
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