search

Wedge-lab / dpclust

Dirichlet Process based methods for subclonal reconstruction of tumours
GNU Affero General Public License v3.0
28 stars 16 forks source link

Incorporate multi-sample core from triplet pipeline #21

Closed sdentro closed 10 months ago

sdentro commented 11 months ago

This PR contains all the differences between regular DPC and the triplet pipeline core functions, such that the triplet pipeline no longer requires any functions to be overloaded.

Have gone through the code line-by-line, this should contain all differences and testing here has not revealed any problems. However, it would be good to verify this independently.

One major change in behaviour is that the pipeline no longer attempts to estimate density for more than 5 samples as that will fail anyway.

Mousiekin commented 9 months ago

Works exactly as described> Gave very similar output to David's recent code that he gave me to run. It refused to work on more than 5 samples and came up with the appropriate error message.