You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
QSnpGATKRecord and QSnpRecord are marked as deprecated for a long time, however, we can't really remove them since they are used by other projects unless we remove the below classes:
QSnpRecord is required by qsnp::PileupPipeline and qsnp::MuTectPipeline requires. This two pipeline seems old and no longer used by recent SNP calls.
QSnpGATKRecord is required by qsnp::util::QJumperWorker, which is also only required by qsnp::PileupPipeline.
qsnp::util::BAMPileupUtil calls both QSnpGATKRecord and QSnpRecord, but this class is not used by qSnp and other projects
qnp::TorrentPipeline is an old method too, even it calls neither QSnpGATKRecord nor QSnpRecord.
Describe the solution you'd like
QSnpGATKRecord and QSnpRecord should be deleted only we decide to remove qsnp::PileupPipeline, qsnp::MuTectPipeline, qnp::TorrentPipeline, qsnp::util::BAMPileupUtil and qsnp::util::QJumperWorker
Additional context
leave for open discussion
The text was updated successfully, but these errors were encountered:
After a discussion, there qsnp::PileupPipeline, qsnp::MuTectPipeline and qnp::TorrentPipeline are old, they should be deprecated now and delete in future.
Is your feature request related to a problem? Please describe.
QSnpGATKRecord and QSnpRecord are marked as deprecated for a long time, however, we can't really remove them since they are used by other projects unless we remove the below classes:
Describe the solution you'd like
QSnpGATKRecord and QSnpRecord should be deleted only we decide to remove qsnp::PileupPipeline, qsnp::MuTectPipeline, qnp::TorrentPipeline, qsnp::util::BAMPileupUtil and qsnp::util::QJumperWorker
Additional context
leave for open discussion
The text was updated successfully, but these errors were encountered: