-
Notifications
You must be signed in to change notification settings - Fork 1
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
Not suitable test data #67
Labels
Comments
When example 2 is embedded into the first spectrum of TP_HeLa_200ng_filtered_pd21 this is the annotated spectrum:
|
When example 1 is embedded into the first spectrum of TP_HeLa_200ng_filtered_pd21 this is the annotated spectrum:
This is the plot of the part: Input part:
Output part:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I firstly assumed that this case was a bug in the code, but as seen in the issue #66, it wasn't.
Problem:
The test data, you can see below, from the deisotoper of the protViz package is not suitable for our deisotoper.
Example 1:
Example 2:
Explanation:
The test data contains only a part of a mass spectrum. Out scoring algorithm uses the whole peaks, that means in a full spectra all, for example 100 up to 400 peaks, not only a tiny part from it. Therefore the score is calculated by using only the given peaks and this results in, that the scores are all 0 (because the score is calculated by comparing the diffs and sums with the amino acids and other atomic masses). Therefore there can't be a specific best path and because we only have one isotopic set in this cases there is only one best path which is chosen randomly, because all paths have the same score. This results in, that you always have different outputs.
Discussion:
We should discuss about better test data or placing the test data into a specific spectrum to have peaks on which the algorithm can rely on.
The text was updated successfully, but these errors were encountered: