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

Show multiple mutations in the PCA plot #92

Closed
gurdeep330 opened this issue Sep 6, 2022 · 3 comments
Closed

Show multiple mutations in the PCA plot #92

gurdeep330 opened this issue Sep 6, 2022 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@gurdeep330
Copy link
Member

@raimondifranc said
"another very helpful update would be giving the possibility to visualize multiple input sequence/mutations on the PCA panel"

@gurdeep330 gurdeep330 added the enhancement New feature or request label Sep 6, 2022
@gurdeep330 gurdeep330 self-assigned this Sep 6, 2022
gurdeep330 pushed a commit that referenced this issue Oct 5, 2022
gurdeep330 pushed a commit that referenced this issue Oct 5, 2022
@maticmarin
Copy link

@gurdeep330 I'm testing this with 15 different sequences and it does run but PCA panel isn't visible. Can you see it or is it just me https://precogxb.bioinfolab.sns.it/output/XWLM8

@gurdeep330
Copy link
Member Author

gurdeep330 commented Oct 18, 2022

Thanks for reporting this @maticmarin. This is arising because precogx internally assigns '_' to the sequence names before processing. I see that this led to some internal conflict and hence the error.

I have opened another issue to address this. Will update you ASAP. A quick way to avoid this, for now, is to replace _ in the sequence name with something else like, say ':'
You can quickly do this by typing: cat input.fasta | tr '_' ':' > newInput.fasta in your terminal. I did this for you already and here is the output.

Looking at the output, another question arose (@maticmarin @raimondifranc). Currently, if you input point mutations, it will display all the mutations (of the same protein) in the PCA plot eg: output. In other words, you must specify the "/" with the Uniprot ID or in the sequence name (if provided input in the FASTA format). I imagined that to be @raimondifranc idea. Or do you guys prefer to have all the inputs (even from different proteins) to be displayed in the PCA plot?

That also means @maticmarin, as of now, you will have to add '/' in the sequence names if you want to view them in one plot. It makes sense only if there are point mutations.

gurdeep330 pushed a commit that referenced this issue Oct 23, 2022
gurdeep330 pushed a commit that referenced this issue Oct 25, 2022
@gurdeep330
Copy link
Member Author

refer #95

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants