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

Are SA tags lifted? #23

Open
DanielNaro opened this issue Jan 19, 2022 · 1 comment
Open

Are SA tags lifted? #23

DanielNaro opened this issue Jan 19, 2022 · 1 comment

Comments

@DanielNaro
Copy link

Hello,
The documentation does not list the SA flag as lifted, and from reading the code I think that it is indeed not the case. Could you confirm this?
Is this a development you are considering? I guess the logic would be fairly similar to ChainMap::lift_segment, would it not?
Thanks and best regards,
Daniel Naro

@milkschen
Copy link
Collaborator

milkschen commented Jan 20, 2022

Hi Daniel,

We don't have a short-term plan to lift the SA flag due to limited bandwidth. The CIGAR information in SA:z is a bit tricky to deal with, though still doable.

As you mentioned, most SAM fields are lifted in ChainMap::lift_segment() for the ChainMap mode. The NM:i and MD:z tags are lifted in void read_and_lift() if the -m optional flag is toggled.

Nae-Chyun

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

No branches or pull requests

2 participants