find_circ icon indicating copy to clipboard operation
find_circ copied to clipboard

Option flank is never used in merge_bed.py

Open ghost opened this issue 7 years ago • 2 comments

I was wondering that the fuzzy search in merge_bed.py gives me the same results as without the fuzzy search. I looked into merge_bed.py and saw that this option is never used. Is there any reason to call the function read_to_hash with parameter flank always with 0 in line 49? inputs = [read_to_hash(a,flank=0) for a in args] Will it work if I replace the zero with options.flank? inputs = [read_to_hash(a,flank=options.flank) for a in args]

ghost avatar Mar 27 '17 15:03 ghost

Hi raistlin91,

well spotted. No, I think you can give it a try. It should work (tm). However, I have not used the fuzzy search in many years. If you could run some tests to make sure it works as expected, I'd be happy to pull a patch from you.

Best regards,

  • Marvin

On Monday, March 27, 2017 11:33:23 AM EDT raistlin91 wrote:

I was wondering that the fuzzy search in merge_bed.py gives me the same results as without the fuzzy search. I looked into merge_bed.py and saw that this option is never used. Is there any reason to call the function read_to_hash with parameter flank always with 0 in line 49? inputs = [read_to_hash(a,flank=0) for a in args] Will it work if I replace the zero with options.flank? inputs = [read_to_hash(a,flank=options.flank) for a in args]

marvin-jens avatar Mar 27 '17 15:03 marvin-jens

Hi @raistlin91

Thanks for the feedback. Just in case you didn't know, the main repository of find_circ moved. So future issues might have higher visibility (also for other users) when opened there.

Also, feel free to ask for features over at ciRcus as we plan to implement (some of) the merge_bed.py functionality there and we need user feedback to prioritize these according to what is actually used.

mschilli87 avatar Mar 27 '17 15:03 mschilli87