Closed sajantanand closed 3 years ago
Merging #14 (02309ec) into main (71873f2) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## main #14 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 7 9 +2
Lines 199 434 +235
==========================================
+ Hits 199 434 +235
Impacted Files | Coverage Δ | |
---|---|---|
src/fitting.py | 100.00% <ø> (ø) |
|
src/cluster.py | 100.00% <100.00%> (ø) |
|
src/preprocess.py | 100.00% <100.00%> (ø) |
|
src/tests/test_cluster.py | 100.00% <100.00%> (ø) |
|
src/tests/test_preprocess.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 71873f2...02309ec. Read the comment docs.
This PR will address several open issues: #6, #7, #10, #11.
For #6, we have renamed the relevant files to 'preprocess.py' and 'test_preprocess.py'. We do this because in addition to featurization of data, we also intend to transform the data so that it has uniform variance. This will most likely be done by PCA.
For #7, #10, #11, we will implement clustering methods using the 'sklearn' library.