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

cooccurrence_matrices/absolute lists wrong SNP #41

Open
ibseq opened this issue Jul 14, 2021 · 7 comments
Open

cooccurrence_matrices/absolute lists wrong SNP #41

ibseq opened this issue Jul 14, 2021 · 7 comments

Comments

@ibseq
Copy link

ibseq commented Jul 14, 2021

Hi DIllon,
I noticed in the cooccurrence_matrices/absolute path that one of the variants has a SNP that is not listed in the mutations.tsv file: specifically , the location is correct but the nucleotide change is wrong: how is this possible and how can I troubleshoot it? I checked many times and re-run it many times. I'm assuming is listed wrong in al other output files.

the snp in question is at position 23012: it has different nctl changes depending on variant. could this be the issue? checked the other variant outputs and they have even different snps in their files, not what I stated in the mutation file.

thanks
ibseq

@dorbarker
Copy link
Owner

Thanks for the bug report. I will look into it.

Could you provide a copy of the mutations file and the cooccurence matrix? You should be able to attach it here, or paste them into gist.github.com

@ibseq
Copy link
Author

ibseq commented Jul 20, 2021

Hi Dillon
any update? - Sent you the two files privately

thanks
ibseq

@dorbarker
Copy link
Owner

Apologies - I was on vacation. I'm back to work on mmmvi now.

However, I don't see any files in my email nor linked anywhere on github.

@ibseq
Copy link
Author

ibseq commented Jul 28, 2021 via email

@dorbarker
Copy link
Owner

Hmm. Weirdly enough, I don't see it - not even accidentally diverted to spam. Sorry about that. Could I trouble you to send it again?

@ibseq
Copy link
Author

ibseq commented Sep 10, 2021 via email

@dorbarker
Copy link
Owner

Hello ibseq,
I just fixed this yesterday. Updating to version 0.10.2 should do the trick. You can confirm your version with mmmvi --version

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