Cleanup suggested by coderabbit

fulcrumgenomics · Nov 11, 2024 · 9addfdc · 9addfdc
1 parent b111c20
commit 9addfdc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/prymer/api/picking.py b/prymer/api/picking.py
@@ -120,9 +120,6 @@ def build_primer_pairs(
     Returns:
         an iterator over all the valid primer pairs, unsorted
     """
-    print(f"Target={target}; lefts={left_primers}.")
-    print(f"Target={target}; rights={right_primers}.")
-
     # Short circuit if we have no left primers or no right primers
     if not any(left_primers) or not any(right_primers):
         return

diff --git a/tests/api/test_picking.py b/tests/api/test_picking.py
@@ -85,6 +85,9 @@ def p(bases: str, tm: float, pos: int, pen: float = 0, chrom: str = "chr1") -> O
 
 def pp(lp: Oligo, rp: Oligo, bases: Optional[str] = None, tm: Optional[float] = None) -> PrimerPair:
     """Generates a primer pair for testing."""
+    if lp.span.end > rp.span.start:
+        raise ValueError("Overlapping primers not supported.")
+
     if bases is None:
         length = rp.span.end - lp.span.start + 1
         needed = length - lp.span.length - rp.span.length