Skip to content

Conversation

@taylorreiter
Copy link
Contributor

updates documentation to include definitions for sourmash gather's csv output

  • Is it mergeable?
  • make test Did it pass the tests?
  • make coverage Is the new code covered?
  • Did it change the command-line interface? Only additions are allowed
    without a major version increment. Changing file formats also requires a
    major version number increment.
  • Was a spellchecker run on the source code and documentation after
    changes were made?

@codecov
Copy link

codecov bot commented Oct 2, 2018

Codecov Report

Merging #553 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #553   +/-   ##
=======================================
  Coverage   88.51%   88.51%           
=======================================
  Files          25       25           
  Lines        3543     3543           
  Branches       37       37           
=======================================
  Hits         3136     3136           
  Misses        407      407

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 544ea1d...7315eaa. Read the comment docs.

@luizirber luizirber merged commit dbb323f into master Oct 3, 2018
@luizirber luizirber deleted the update/doc_tr branch October 3, 2018 19:31
+ intersect_bp: the approximate number of base pairs in common between the query and the match
+ f_orig_query: fraction of original query; the fraction of the original query that is contained within the match
+ f_match: fraction of match; the fraction of the match that is contained within the query
+ f_unique_to_query: fraction unique to query; the fraction of the query that uniquely overlaps with the match
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comment: this is unique with respect to all previously output matches in the gather output.

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.

4 participants