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

Render cut site when it goes over the end of the sequence #272

Closed
guzmanvig opened this issue Sep 24, 2024 · 2 comments · Fixed by #273
Closed

Render cut site when it goes over the end of the sequence #272

guzmanvig opened this issue Sep 24, 2024 · 2 comments · Fixed by #273
Assignees

Comments

@guzmanvig
Copy link
Collaborator

guzmanvig commented Sep 24, 2024

Mentioned in:

// TODO: remove this exclusion of cut-sites that cross the zero index after even more

And in: # 270

Describe the bug

When an enzyme's combined recognition site and cut site sequence span straddles the last SeqBlock and first SeqBlock. This feature is not implemented in Seqviz as the TODO in the code mentions

Expected behavior

If an enzyme's recognition sequence is detected near the end of the final SeqBlock and some-or-all the associated cut site is offset from it such that it extends off the end of the final SeqBlock, then the cut site should be displayed on the first SeqBlock, at the top of the sequence viewer display.

@guzmanvig guzmanvig self-assigned this Sep 24, 2024
@jpsorensen-asimov
Copy link

https://github.com/guzmanvig is this one you're actively working on? Was considering having a member of our team attempt to attack this one and contribute a fix, but don't want to have multiple people chasing this if you all were already on the case!

If we do tackle it, a desired behavior question - are you imagining that a recognition site would be rendered even if the recognition site went over the end of a circular sequence? Meaning part of the recognition site is on one side and part of the recognition site is on the other side? Or were you just imagining that we'd want to handle the case where the recognition site is entirely on one side of the boundary, but the cut site is on the other side? Maybe it's not a meaningful difference with the way SeqViz currently defines recognition sites (padded out with Ns to include the cut sites), which would necessitate the former. But wanted to see if you had a clear opinion based on the SeqViz philosophy.

Thanks!

@guzmanvig
Copy link
Collaborator Author

@jpsorensen-asimov I am indeed already working on this. I'm not 100% sure this will answer your question but the desired functionality imo is that Seqviz will render the recognition site (+ cut site) in both scenarios (and I add a third):

  • If the recognition site is at the end of the sequence and the cut site at the start
  • If the recognition site is partially at the end and partially at the start
  • If the recognition site is at the end and the cut site is partially at the end and partially at the start.

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.

2 participants