Skip to content

Commit

Permalink
Ultima.flow annotations.fix (#8442)
Browse files Browse the repository at this point in the history
* hmer ondel must have mon length

* Revert "hmer ondel must have mon length"

This reverts commit 7852871.

* remove superfluous variant type condition

* fix error message to actually reflect missing argument

* fixed unittest to include variant type

* Remove conflict
  • Loading branch information
dror27 committed Nov 13, 2023
1 parent 2900e01 commit 683eaa8
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import htsjdk.variant.variantcontext.VariantContext;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.broadinstitute.hellbender.cmdline.StandardArgumentDefinitions;
import org.broadinstitute.hellbender.engine.ReferenceContext;
import org.broadinstitute.hellbender.exceptions.GATKException;
import org.broadinstitute.hellbender.tools.walkers.annotator.InfoFieldAnnotation;
Expand Down Expand Up @@ -149,7 +150,7 @@ private String establishReadGroupFlowOrder(final LocalContext localContext, fina
// if here, no flow order was found. may we use a default?
if ( isActualFlowOrderRequired() ) {
localContext.generateAnnotation = false;
flowMissingOneShotLogger.warn("this.getClass().getSimpleName() + \" annotation will not be calculated, no '\" + StandardArgumentDefinitions.FLOW_ORDER_FOR_ANNOTATIONS + \"' argument provided\"");
flowMissingOneShotLogger.warn(this.getClass().getSimpleName() + " annotation will not be calculated, no '" + StandardArgumentDefinitions.FLOW_ORDER_FOR_ANNOTATIONS + "' argument provided");
}

return FlowBasedRead.DEFAULT_FLOW_ORDER;
Expand Down

0 comments on commit 683eaa8

Please sign in to comment.