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

Expose option to set kmer_probe_map_k parameter #16

Merged
merged 3 commits into from Jul 18, 2018

Conversation

haydenm
Copy link
Collaborator

@haydenm haydenm commented Jul 18, 2018

This adds a --kmer-probe-map-k argument, which specifies the k-mer length used when constructing a map of k-mers to probes. (In particular, CATCH tries to determine a suitable k-mer length using this value as a lower bound; if it cannot, it takes a randomized approach and uses this value as the k-mer length.)

This is useful when designing short probes (e.g., meant to be primers), in which case the default k-mer length may be too short relative to the probe to find mappings of the probes to the target sequence.

The two different tests mistakenly had the same name.
This provides varied values of the k-mer length used to construct a map of
k-mers to probes. Each of these maps (for each k) is then used to test the
probe.find_probe_covers_in_sequence() function.
This parameter specifies the k-mer length used when constructing a map of
k-mers to probes.  (In particular, CATCH tries to determine a suitable k-mer
length using this value as a lower bound; if it cannot, it takes a randomized
approach and uses this value as the k-mer length.)

This is useful when designing short probes (e.g., meant to be primers), in
which case the default k-mer length may be too short relative to the probe to
find mappings of the probes to the target sequence.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 94.787% when pulling f8b5a6c on expose-kmer-probe-map-k into af7929e on master.

@haydenm haydenm merged commit 5777f1b into master Jul 18, 2018
@haydenm haydenm deleted the expose-kmer-probe-map-k branch July 18, 2018 18:13
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.

None yet

2 participants