miRTop / mirtop

command lines tool to annotate miRNAs with a standard mirna/isomir naming
https://mirtop.readthedocs.org
MIT License
18 stars 21 forks source link

integrate sqlite database #59

Closed arunhpatil closed 4 years ago

codecov[bot] commented 4 years ago

Codecov Report

Merging #59 into master will increase coverage by 0.57%. The diff coverage is 84.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage   62.99%   63.57%   +0.57%     
==========================================
  Files          40       42       +2     
  Lines        3748     3846      +98     
==========================================
+ Hits         2361     2445      +84     
- Misses       1387     1401      +14
Impacted Files Coverage Δ
test/test_automated_analysis.py 92.09% <ø> (+0.25%) :arrow_up:
test/test_functions.py 89.41% <100%> (+0.42%) :arrow_up:
mirtop/convert/__init__.py 100% <100%> (ø)
mirtop/convert/format.py 81.08% <81.08%> (ø)

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 7912ad5...8386e99. Read the comment docs.

lpantano commented 4 years ago

Hi, Thanks for this!

you are going to kill me...but the PR should be toward dev. That means that probably there is some code that is not up to date.

Can you start from the current dev branch, make the changes there and start a new PR toward dev not master please?