Closed adbailey4 closed 7 years ago
Merging #40 into master will increase coverage by
9.24%
. The diff coverage is77.11%
.
@@ Coverage Diff @@
## master #40 +/- ##
==========================================
+ Coverage 16.69% 25.93% +9.24%
==========================================
Files 9 9
Lines 1084 1226 +142
==========================================
+ Hits 181 318 +137
- Misses 903 908 +5
Impacted Files | Coverage Δ | |
---|---|---|
nanotensor/run_nanotensor.py | 0% <ø> (ø) |
:arrow_up: |
nanotensor/utils.py | 37.19% <100%> (+10.07%) |
:arrow_up: |
nanotensor/data_preparation.py | 81.75% <75.27%> (+5.89%) |
:arrow_up: |
nanotensor/error.py | 59.25% <83.33%> (+6.87%) |
:arrow_up: |
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 05ca7a4...c99a17b. Read the comment docs.
data_preparation.py
- Added quite a bit of error handling as well as implementing the deepnano labeling scheme. The version they were using was with fast5 files from an older chemistry which meant that some of the steps had to approximated. Also, instead of implementing their exact algorithm for creating alignment calls, I introduced my own heuristic for determining which events are labelled and which are not. We will see what works.error.py
- Added another Error for catching specific errors indata_preparation.py
run_nanotensor.py
- Added a small note in one of the methods for determining memory usagedata_preparation_test.py
- Added a number of tests and some more indepth testing to keep track of many of the features that are currently in placeutils_test.py
- Added check_duplicate_characters testutils.py
- Edited sum_to_one method. This may be a bottleneck for analysis so we may need to identify a faster method.Moved pickle files to test_files/data_prep_test_files Removed files no-longer being referenced Added a very short tsv test file to increase speed of our tests