Skip to content

Commit

Permalink
Merge branch 'jdaw/skip-barcode-trimming-for-polya' into 'master'
Browse files Browse the repository at this point in the history
skip barcode trimming when running polyA estimation

Closes DOR-602

See merge request machine-learning/dorado!873
  • Loading branch information
tijyojwad committed Mar 7, 2024
2 parents 851023f + f71ccf4 commit e8fb085
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions dorado/cli/basecaller.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -423,8 +423,8 @@ int basecaller(int argc, char* argv[]) {
.default_value(std::nullopt);
parser.visible.add_argument("--estimate-poly-a")
.help("Estimate poly-A/T tail lengths (beta feature). Primarily meant for cDNA and "
"dRNA use cases. Note that if this flag is set, then adapter/primer detection "
"will be disabled.")
"dRNA use cases. Note that if this flag is set, then adapter/primer/barcode "
"trimming will be disabled.")
.default_value(false)
.implicit_value(true);
parser.visible.add_argument("--poly-a-config")
Expand Down Expand Up @@ -535,10 +535,10 @@ int basecaller(int argc, char* argv[]) {
"if you are also using --estimate-poly-a.");
std::exit(EXIT_FAILURE);
}
no_trim_primers = no_trim_adapters = true;
no_trim_primers = no_trim_adapters = no_trim_barcodes = true;
spdlog::info(
"Estimation of poly-a has been requested, so adapter/primer trimming has been "
"disabled.");
"Estimation of poly-a has been requested, so adapter/primer/barcode trimming has "
"been disabled.");
polya_config = parser.visible.get<std::string>("--poly-a-config");
}

Expand Down

0 comments on commit e8fb085

Please sign in to comment.