[Biopython-dev] 1.62b test coverage report

[Peter Cock]

On Sun, Aug 18, 2013 at 7:14 PM, Eric Talevich <eric.talevich at gmail.com> wrote:
> On Fri, Aug 16, 2013 at 2:31 AM, Peter Cock wrote:
>>
>> Well that API break is a bit annoying, but your test changes make sense.
>>
>> Do we need to add these new switches to the wrapper itself?
>
>
> Here's the commit to add those switches to the wrapper:
> https://github.com/biopython/biopython/commit/cc234b75e6e82cf9f51e3384a4fbfa1e888a3af1
>
> I suppose it would be helpful if the wrapper detected the version of Prank
> and handled the show(tree|xml) flags appropriately to avoid errors. But that
> would require running the executable first, I think, which is not something
> our wrappers normally do. (And then it would make sense to cache the result
> for the duration of the running process.)
>
> -Eric

Historically we've just documented this kind of issue in the
parameter docstring - the idea of auto-running the tool in
the background to check the version just sounds like Trouble.

Peter