Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pairToBed writes 1 BedPE line per match in either mode #1023

Open
JoeVieira opened this issue Nov 3, 2022 · 1 comment · May be fixed by #1026
Open

pairToBed writes 1 BedPE line per match in either mode #1023

JoeVieira opened this issue Nov 3, 2022 · 1 comment · May be fixed by #1026

Comments

@JoeVieira
Copy link

When running pairToBed in output BedPE mode and type = either ( default)
a match on either read will output a BedPE entry line
a match on both reads will output 2 BedPE entries.

This seems logically inconsistent with the purpose of the tool, additionally it's not consistent with other match types.
For instance in type = neither. When 0 matches occur only 1 entry is output.

In Bam output mode this behaves as expected. Both read records are output consistently.

If you agree this is a logical bug, I'm happy to contribute a fix.

To be logically consistent it seems the result should never include multiple identical bedpe entry lines, because bedpe entry already encodes both pairs from the original paired input file.

@JoeVieira JoeVieira linked a pull request Nov 14, 2022 that will close this issue
@JoeVieira
Copy link
Author

JoeVieira commented Jan 3, 2023

@arq5x Any thoughts on how i can move my fix for this forward. I contributed a PR already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant