-
Notifications
You must be signed in to change notification settings - Fork 0
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
Variants shouldn't be map to reference sequences #1
Comments
Sounds great - I'll see if I can plan out another edit to implement that this week. Hey also, James passed a maven command on to me but I am still facing errors when running pepgenome from the terminal. Would you have any ideas? |
Which error do you get? |
this command should work:
|
Hey, sorry for delay - job hunting at the minute since my placement contract is ending. I'm having trouble actually generating the JAR. The JAR wasn't working when generated from the Iintellij Jjava application configurations that I was using for debugging. So, I switched it to a maven configuration and I'm trying to get it to build. I tried running "mvn install" on the command line which downloaded some files but resulted in: [INFO] ------------------------------------------------------------------------ Everything is set to use the same version of java, any idea what's causing this? I didn't get this error during the testing stages at all. |
I remove that dependency. Can you try now @EamonnOCearnaigh |
[INFO] ------------------------------------------------------------------------ |
I'm running "mvn install" on the command line. Is that the right command? |
Yes, |
I have added the dependency to our internal maven repo. Can you try again @EamonnOCearnaigh ? |
It got further this time. Downloaded from central: https://repo.maven.apache.org/maven2/org/apache/struts/struts-core/1.3.8/struts-core-1.3.8.jar (329 kB at 270 kB/s) Also my Java version is: java version "13.0.2" 2020-01-14 |
Java doesn't find thw javadoc command. I have not idea why. Can you check in stackoverflow |
Will do. For now, do you have a working JAR you could post to GitHub or send by email? We just need to place it into a NextFlow pipeline. |
@EamonnOCearnaigh
As I mentioned in the discussion the other day would be great if we have a logic in the tool that if the user uses the MM options
-mm NUM
to allow missmatches and the query peptide matches with 0 miss-matches and 1..2.. etc miss-matches have a way to discard the 1..2. .etc options. The use case is:Peptide Query: AAAA -> Protein A in sequence ..AAAA... and Protein B in sequence ..AVAA.. we discard the second match because it has more probabilities (I will say 100%) that this is only a reference peptide. James mentioned that he is doing a two-step search to discard reference peptides first and then search with multiple gaps.
Would be great if we can implement that feature.
The text was updated successfully, but these errors were encountered: