Closed aarmey closed 2 years ago
The results will obviously vary with seqWeight and cluster number. Perhaps we go with whatever we settle on for the rest of the paper?
Merging #512 (d2138d3) into master (95305ea) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #512 +/- ##
=======================================
Coverage 38.68% 38.68%
=======================================
Files 16 16
Lines 1357 1357
=======================================
Hits 525 525
Misses 832 832
Flag | Coverage Δ | |
---|---|---|
unittests | 38.68% <100.00%> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
msresist/clustering.py | 51.33% <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 95305ea...d2138d3. Read the comment docs.
Great thanks! I used 30 clusters and a weight of 100. I removed cluster 19 because for some reason there was only one peptide in this cluster. This seemed to happen all the time with 25-35 clusters.
I just added the x and y tick labels and set the DDMC parameters. Is this ready to merge?
Sure!
@mcreixell this implements issue #508