Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for MapMyGenome #29

Merged
merged 7 commits into from
Oct 17, 2019
Merged

Add support for MapMyGenome #29

merged 7 commits into from
Oct 17, 2019

Conversation

willgdjones
Copy link
Contributor

@willgdjones willgdjones commented Oct 12, 2019

This PR adds support for MapMyGenome data files.

@codecov-io
Copy link

Codecov Report

Merging #29 into develop will increase coverage by 0.87%.
The diff coverage is 81.81%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #29      +/-   ##
===========================================
+ Coverage    88.76%   89.63%   +0.87%     
===========================================
  Files            5        5              
  Lines         1041     1052      +11     
  Branches       194      196       +2     
===========================================
+ Hits           924      943      +19     
+ Misses          72       62      -10     
- Partials        45       47       +2
Impacted Files Coverage Δ
src/snps/io.py 85.23% <81.81%> (-0.12%) ⬇️
src/snps/resources.py 93.92% <0%> (+4.04%) ⬆️

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 9b543fa...1acb3ca. Read the comment docs.

@apriha
Copy link
Owner

apriha commented Oct 17, 2019

Thanks again @willgdjones ! Merging PR.

@apriha apriha merged commit ed953b1 into apriha:develop Oct 17, 2019
apriha pushed a commit that referenced this pull request Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants