search

deeptools / deepTools

Tools to process and analyze deep sequencing data.
Other
684 stars 212 forks source link

Maturin #1336

Closed WardDeb closed 2 days ago

WardDeb commented 2 days ago

WIP for deepTools 4 overhaul.

bgruening commented 2 days ago

@WardDeb would it make sense to split up this PR in many small ones? Maybe even per tool? I guess this makes it easier to review them.

WardDeb commented 2 days ago

@WardDeb would it make sense to split up this PR in many small ones? Maybe even per tool? I guess this makes it easier to review them.

Not so sure to split them up atm (note that for this PR only bamCoverage is properly implemented, only babysteps for bamCompare, so this PR corresponds to one tool..). The number of files changed looks daunting as all the python code got moved into pydeeptools folder, but actually not that much changed.

I'll go ahead and merge this, but keep maturin separated for now. We can discuss moving up to develop next wednesday :)