Skip to content

Fix contrib API#70

Merged
standage merged 3 commits intomasterfrom
contrib
Nov 5, 2019
Merged

Fix contrib API#70
standage merged 3 commits intomasterfrom
contrib

Conversation

@standage
Copy link
Member

@standage standage commented Nov 5, 2019

While updating a demo of the Python API, I discovered that the microhapulator.contrib.contrib has no way of processing a Profile object already in memory. It could only take a JSON file or BAM/FASTA as input. This update fixes the API so that the microhapulator.contrib.contrib works only on Profile objects, and changes the CLI to accommodate all the requisite handling of input files.


  • Changes are clearly described above
  • Any relevant issue threads are referenced in the description
  • Any new features are tested (see docs/DEVEL.md for details)
  • Substantial changes are documented in CHANGELOG.md (see https://keepachangelog.com/en/1.0.0/)

@codecov-io
Copy link

codecov-io commented Nov 5, 2019

Codecov Report

Merging #70 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
+ Coverage   96.24%   96.25%   +0.01%     
==========================================
  Files          25       25              
  Lines         798      802       +4     
  Branches      157      157              
==========================================
+ Hits          768      772       +4     
  Misses         16       16              
  Partials       14       14
Impacted Files Coverage Δ
microhapulator/contrib.py 100% <100%> (ø) ⬆️

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 1daceb4...c991a4b. Read the comment docs.

@standage standage merged commit 51b017c into master Nov 5, 2019
@standage standage deleted the contrib branch November 5, 2019 21:45
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.

2 participants