[Biopython-dev] Bug #3109: Record class in Bio.SCOP.Cla has hierarchy member as list instead of dictionary

Peter biopython at maubp.freeserve.co.uk
Wed Aug 11 06:29:32 EDT 2010


On Fri, Aug 6, 2010 at 9:09 PM, Jeffrey Finkelstein
<jeffrey.finkelstein at gmail.com> wrote:
> I have submitted a bug report and a patch for a current feature of the
> Bio.SCOP.Cla module that makes using it somewhat difficult. Specifically,
> the Bio.Scop.Cla.Record class has a "hierarchy" member which is currently a
> list, but should be a dictionary, according to the SCOP parseable
> classification file format "specification" (it is an informal specification)
> here:
> http://scop.mrc-lmb.cam.ac.uk/scop/release-notes.html#scop-parseable-files.
>
> For a usage example, see the comments I have made at:
> http://bugzilla.open-bio.org/show_bug.cgi?id=3109
>
> I have CC'ed the original author of the module. Gavin (or anyone else), do
> you have any objections to this change?
>
> Jeffrey

As I commented on the bug, I'm happy with this change in principle,
except for the fact it breaks backwards compatibility. If we ask on the
main list and there are no objections, then the code proposed looks
fine:

http://github.com/jfinkels/biopython/commit/e52255f06c08aad1556eb518e2e0da8f030765ff

Peter


More information about the Biopython-dev mailing list