[Bioperl-l] TextResultWriter

ClarkeW ClarkeW at AGR.GC.CA
Tue May 22 21:10:08 UTC 2007


Hi, 

I am interested in becoming a bioperl developer as I have recently found a
bug in TextResultWriter. I know that I should submit the bug fixes using the
protocol outlined in the How To but I haven't been able to login to the CVS
anonymously to check it out. However, I have checked that the bug still
exists in the most recent version of the code using the web interface to the
CVS repositories. The bug is between lines 433 and 443, and deals with the
reporting of the number of letters in the database and the number of entries
in the database. My fix would be to change the existing code block:

from:

    Number of letters in database: %s
    Number of sequences in database: %s

Matrix: %s
},
        $result->database_name(),
        $result->get_statistic('posted_date') ||
        POSIX::strftime("%b %d, %Y %I:%M %p",localtime),
        &_numwithcommas($result->database_entries()),
        &_numwithcommas($result->database_letters()),
        $result->get_parameter('matrix') || '');

to: 

    Number of letters in database: %s
    Number of sequences in database: %s

Matrix: %s
},
        $result->database_name(),
        $result->get_statistic('posted_date') ||
        POSIX::strftime("%b %d, %Y %I:%M %p",localtime),
        &_numwithcommas($result->database_letters()),
        &_numwithcommas($result->database_entries()),
        $result->get_parameter('matrix') || '');

I believe that this is a simple enough modification that it does not require
any new test cases.

Cheers, Wayne




More information about the Bioperl-l mailing list