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

MergeMotifsAcrossRuns.ipynb should account for indexes with respect to --window #48

Open
KnightAsterial opened this issue Mar 4, 2024 · 0 comments

Comments

@KnightAsterial
Copy link

Extension of #20, MergeMotifsAcrossRuns.ipynb creates a new seqlet set that combines the results from two different modisco runs. However, it does not take into account that tfmodisco-lite trims each input sequence around the center with respect to --window and labels start and end using the trimmed sequences. Thus, the recreated seqlets do not match the original seqlets.

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

1 participant