<html>
<head>
<style>
body {
  font-family: Verdana, sans-serif;
  font-size: 0.8em;
  color:#484848;
}
h1, h2, h3 { font-family: "Trebuchet MS", Verdana, sans-serif; margin: 0px; }
h1 { font-size: 1.2em; }
h2, h3 { font-size: 1.1em; }
a, a:link, a:visited { color: #2A5685;}
a:hover, a:active { color: #c61a1a; }
a.wiki-anchor { display: none; }
fieldset.attachments {border-width: 1px 0 0 0;}
hr {
  width: 100%;
  height: 1px;
  background: #ccc;
  border: 0;
}
span.footer {
  font-size: 0.8em;
  font-style: italic;
}
</style>
</head>
<body>
Issue #2968 has been updated by Peter Cock.

<ul>
  <li><strong>Description</strong> updated (<a title="View differences" href="https://redmine.open-bio.org/journals/diff/15422?detail_id=1798">diff</a>)</li>
  <li><strong>Status</strong> changed from <i>New</i> to <i>Closed</i></li>
  <li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li>
</ul>

<p>I'm going to mark this as closed - the original git commit has gone, and it seems this work had some influence on the main reposoitory already. Thanks!</p>
<hr />
<h1><a href="https://redmine.open-bio.org/issues/2968#change-15422">Bug #2968: Modifications to Emboss eprimer3 parser and associated files</a></h1>

<ul><li>Author: Leighton Pritchard</li>
<li>Status: Closed</li>
<li>Priority: Normal</li>
<li>Assignee: Biopython Dev Mailing List</li>
<li>Category: Main Distribution</li>
<li>Target version: 1.52</li>
<li>URL: </li></ul>

<p>The existing Emboss primer3/eprimer3 code has a couple of issues, and some scope for improvement:</p>


        <p>- The existing Primer3.py parser code can only parse output when eprimer3 is applied to a single sequence.  When eprimer3 is applied to multiple sequence input, it groups all primers for all sequences into a single record, which may incorrectly associate primers with the wrong sequences in downstream analysis.<br />- The current parser lacks an iterator for iterating over multiple sequence output<br />- The current parser creates 'ghost' primers for all primer pairs, with length zero and sequence as an empty string; it does not do this for internal oligos.  A more intuitive solution might be to return None for absent primers/oligos<br />- The current data model stores all primer data as individual attributes.  It might be more useful to group the attributes of individual primers into their natural associations</p>


        <p>I have written new code for Emboss/Primer3.py that adds iterator/multiple sequence parsing functionality to the parser, and extensively revises the object model for the data.  The Record and Primers objects are retained, but each primer/oligo is now represented by a Primer object that collects the relevant data together.  The Record object has a new attribute that allows the sequence to be recorded directly, rather than having to be parsed from the comments attribute.  The new data model retains the old attribute-based access for compatibility, but adds direct access to the Primer objects (where present) by .forward, .reverse and .oligo attributes, and by keywords.</p>


        <p>One change was required to the unit test, to account for the reporting of absent primers as None, rather than having 'null' attributes.  I've added two further test output files, which may be rather large for the distribution (60kb total), and doctests that use these.</p>


        <p>The code can be inspected at my GitHub repository:</p>


        <p><a class="external" href="http://github.com/widdowquinn/biopython/commit/b4701079ced297d7af5aa75b46738280c8783fe0">http://github.com/widdowquinn/biopython/commit/b4701079ced297d7af5aa75b46738280c8783fe0</a></p>


        <p>This enhancement request also relates to bug 2966.</p>



<hr />
<span class="footer"><p>You have received this notification because you have either subscribed to it, or are involved in it.<br />To change your notification preferences, please click here and login: <a class="external" href="http://redmine.open-bio.org">http://redmine.open-bio.org</a></p></span>
</body>
</html>