twolinin / longphase

GNU General Public License v3.0
99 stars 9 forks source link

Suggestion: Address README to accomodate for Sniffles 2 #7

Closed tuannguyen8390 closed 2 years ago

tuannguyen8390 commented 2 years ago

Hi dev,

Just a suggestion :), Sniffles 2 was released a few days ago, the --num_reads_report -1 flag is now changed to --output-rnames. See #288

I am able to phase my data using LongPhase with Sniffles 2. So I presume it would be nice if you can update the README to include the new flag.

Many thanks,

Tuan

ythuang0522 commented 2 years ago

Thank you for the information. The usage of sniffles2 has been added in README.

tuannguyen8390 commented 2 years ago

Awesome, thanks :) - Tuan