search

wdecoster / surpyvor

A python wrapper around SURVIVOR
MIT License
19 stars 1 forks source link

about surpyvor highconf #17

Closed prasundutta87 closed 3 years ago

prasundutta87 commented 3 years ago

Hi,

I observed that surpyvor highconf that technically mimics surpyvor merge (with the option SV calls should be supported by all callers) does not take strand into account and the options to change that, is not present in surpyvor highconf.

I tested the above statement using SURVIVOR merge where I put the option of "They should agree on strand" to no.

Can the strand option be added to surpyvor highconf or has it not been put due to some reason?

Regards, Prasun

wdecoster commented 3 years ago

Hi Prasun,

Thanks or the suggestion, I'll try to add it soon.

Cheers, Wouter

prasundutta87 commented 3 years ago

Thanks a lot @wdecoster !!

wdecoster commented 3 years ago

Surpyvor v0.10.0 now has a --strand flag for highconf. Please let me know if you have other suggestions or feedback.

Best, Wouter

prasundutta87 commented 3 years ago

Thanks a lot for this quick action! :-)