Skip to content

Commit

Permalink
Make giraffe build long read minimizers for long read presets
Browse files Browse the repository at this point in the history
  • Loading branch information
xchang1 committed Nov 25, 2024
1 parent ae25abf commit 1c97950
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/index_registry.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4080,6 +4080,7 @@ IndexRegistry VGIndexes::get_vg_index_registry() {
const IndexGroup& constructing) {
if (IndexingParameters::verbosity != IndexingParameters::None) {
cerr << "[IndexRegistry]: Constructing minimizer index and associated zipcodes." << endl;
cerr << "\tuse parameters -k " << IndexingParameters::minimizer_k << " -w " << IndexingParameters::minimizer_w << (IndexingParameters::minimizer_W ? " -W " : "") << endl;
}

// TODO: should the distance index input be a joint simplification to avoid serializing it?
Expand Down
7 changes: 7 additions & 0 deletions src/subcommand/giraffe_main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1367,6 +1367,13 @@ int main_giraffe(int argc, char** argv) {
} else {
// Apply the preset values.
found->second.apply(*parser);

if (param_preset == "hifi" || param_preset == "r10") {
//Set the minimizer parameters for the index registry to make minimizers for long reads
IndexingParameters::minimizer_k=31;
IndexingParameters::minimizer_w=50;
IndexingParameters::minimizer_W=true;
}
}
}
break;
Expand Down

1 comment on commit 1c97950

@adamnovak
Copy link
Member

Choose a reason for hiding this comment

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

vg CI tests complete for branch autoindex-zipcodes. View the full report here.

16 tests passed, 0 tests failed and 0 tests skipped in 17555 seconds

Please sign in to comment.