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

POA end gap/semi-global alignment #568

Open
ctsa opened this issue Jan 2, 2024 · 7 comments
Open

POA end gap/semi-global alignment #568

ctsa opened this issue Jan 2, 2024 · 7 comments

Comments

@ctsa
Copy link

ctsa commented Jan 2, 2024

Hi, Nice to see recent progress on banded POA! Is there any support for some kind of clipping/end-gap feature for POA to allow semi-global/overlap alignments to the graph?

cc: @lorewar2

@lorewar2
Copy link
Member

lorewar2 commented Jan 4, 2024

Hi, The current code does not include clipping, but I am sure that it can be integrated. I'll take a look and try to figure something out. Thanks.

@ctsa
Copy link
Author

ctsa commented Jan 4, 2024

Thanks! I wasn't sure whether this would need something like a seq to graph version of LCSk++ to setup (I'm only familiar with the rust bio pairwise banding method), would be great if there's a clipping scheme that can be worked out for the POA version.

@lorewar2
Copy link
Member

lorewar2 commented Jan 6, 2024

Yes, but LCSk++ was not used for the POA banding; the max scoring cell in the current row was considered as the band center for the next row, and so on. I'll try to implement unbanded POA clipping first (semi-global, local, etc).

@lorewar2
Copy link
Member

lorewar2 commented Jan 6, 2024

I see the problem with working with banded POA now,

@ctsa
Copy link
Author

ctsa commented Apr 3, 2024

I just noticed this looks like it's in the latest release, thanks!

@ctsa
Copy link
Author

ctsa commented Apr 4, 2024

Will be great to see if the same capability can be worked into the banded POA eventually. This is already a big step ahead though.

@lorewar2
Copy link
Member

lorewar2 commented Apr 5, 2024

I am trying to extend LCSk++ to work with POA, but I am not sure if this will work correctly.

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