Closed someshsingh22 closed 4 years ago
Resolves #15
Merging #18 into master will increase coverage by
8.92%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #18 +/- ##
==========================================
+ Coverage 79.16% 88.09% +8.92%
==========================================
Files 4 5 +1
Lines 48 84 +36
==========================================
+ Hits 38 74 +36
Misses 10 10
Impacted Files | Coverage Δ | |
---|---|---|
decepticonlp/metrics/char_metrics.py | 100.00% <100.00%> (ø) |
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 c93cde8...cc4c9fc. Read the comment docs.
Ok sounds good. Put tests in tests
though
Would be better if we follow the same directory structure in tests. So yours will be in tests/test_metrics/....py
I dont get you @rajaswa, Where have we done the unit testing before ? I have tested all functionalities up to metrics from the root dir. there is no .tests rn I guess
He meant tests
I guess.
Ok, I actually don't know how that works, any link ?
What?
Update: Metrics are also added, for floating-point errors absolute error is checked with a tolerance of 1e-5
Levenshtein Edit Distance Added with polynomial complexity (numpy optimized)
Jaccard Distance Done
Euclidean Distances complete