[Biopython-dev] [Bug 2949] New: _parse_pdb_header_list: REVDAT is for oldest entry.

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=2949

           Summary: _parse_pdb_header_list: REVDAT is for oldest entry.
           Product: Biopython
           Version: 1.52
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main Distribution
        AssignedTo: biopython-dev at biopython.org
        ReportedBy: TallPaulInJax at yahoo.com


Hi,

I don't know if this is considered a bug or a feature, but currently
_parse_pdb_header_list in parse_pdb_header.py is grabbing the least recent date
when I believe it should be grabbing the MOST current date.

Here is the current code:
        elif key=="REVDAT":
            rr=re.search("\d\d-\w\w\w-\d\d",tail)
            if rr!=None:
                dict['release_date']=_format_date(_nice_case(rr.group()))

And here is the fix, with additional REVDAT components added (can't hurt! :-)
):
        elif key=="REVDAT":
            #Modified by Paul T. Bathen to get most recent date instead of
oldest date.
            #Also added additional dict entries
            if dict['release_date'] == "1909-01-08": #set in init
                rr=re.search("\d\d-\w\w\w-\d\d",tail)
                if rr!=None:
                    dict['release_date']=_format_date(_nice_case(rr.group()))

                dict['mod_number'] = hh[7:10].strip()
                dict['mod_id'] = hh[23:28].strip()
                dict['mod_type'] = hh[31:32].strip()

Paul


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.