Re: [PATCH] Add Ribotaper.

2016-07-10 Thread Ben Woodcroft
On 10/07/16 05:43, Ricardo Wurmus wrote: Ben Woodcroft writes: The patch LGTM otherwise, except that I would unpack 'ORF' and use the term 'ribo-seq' in the description. I've never looking at ribo-seq data before and I'm not familiar, so I wasn't sure what 'ribosome profiling' meant exactly

Re: [PATCH] Add Ribotaper.

2016-07-09 Thread Ricardo Wurmus
Ben Woodcroft writes: >> I think in the ribotaper/bedtools case we should just name the variable >> “bedtools-2.18” and make it public. > > I agree with should call it 'bedtools-2.18' and make it public (but with > a comment explaining why the old version is kept), just as in e.g. > 'python-f

Re: [PATCH] Add Ribotaper.

2016-07-08 Thread Ben Woodcroft
On 07/07/16 23:45, Ricardo Wurmus wrote: Ben Woodcroft writes: On 07/07/16 19:26, Ricardo Wurmus wrote: Hi Guix, this patch adds a bioinformatics tool called "Ribotaper". It needs a somewhat older version of bedtools, because the output format produced by bedtools changed after version 2.

Re: [PATCH] Add Ribotaper.

2016-07-07 Thread Ricardo Wurmus
Ben Woodcroft writes: > On 07/07/16 19:26, Ricardo Wurmus wrote: >> Hi Guix, >> >> this patch adds a bioinformatics tool called "Ribotaper". It needs a >> somewhat >> older version of bedtools, because the output format produced by bedtools >> changed after version 2.18.0 (they don't seem to c

Re: [PATCH] Add Ribotaper.

2016-07-07 Thread Ben Woodcroft
On 07/07/16 19:26, Ricardo Wurmus wrote: Hi Guix, this patch adds a bioinformatics tool called "Ribotaper". It needs a somewhat older version of bedtools, because the output format produced by bedtools changed after version 2.18.0 (they don't seem to care about semantic versioning). This is