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

perform rescue after transfer #93

Closed
pavel-shliaha opened this issue Jul 25, 2015 · 7 comments
Closed

perform rescue after transfer #93

pavel-shliaha opened this issue Jul 25, 2015 · 7 comments
Assignees
Labels
Milestone

Comments

@pavel-shliaha
Copy link
Collaborator

As discussed with Sebastian on 25.07.15 there is currently a bug with rescue functionality. It seems that if or

filterUniqueMatches
filterNonUniqueMatches

are used and no EMRT has been transferred after them (i.e. either not enough fragments are present after filterUniqueMatches or not enough difference is present in filterNonUniqueMatches), then no EMRT is assigned to a peptide by synapter. But at the same time no rescue is performed, since rescue is performed before filtering on fragments functionality.

To solve this rescue needs to always be done after all filtering as the last step of synapter analysis

@sgibb sgibb added this to the 2.0 milestone Jul 25, 2015
@sgibb sgibb self-assigned this Jul 25, 2015
@sgibb
Copy link
Collaborator

sgibb commented Jul 25, 2015

I introduced rescueEMRTs (and the corresponding code in findEMRTs is removed). You have to call it manually now, e.g.:

# [...]
findEMRTs(synapterAnalysis)
# [...] further filter, e.g. fragment matching etc.
rescueEMRTs(synapterAnalysis)
# or: 
rescueEMRTs(synapterAnalysis, method="copy")

@sgibb sgibb closed this as completed Jul 25, 2015
@pavel-shliaha
Copy link
Collaborator Author

Unfortunately still problems. I have run the following code with the files in Z:\RAW\pvs22_QTOF_DATA_data3\data_for_synapter_2.0\bug\2015_07_19_problems_with_loading_spectrum\for_synapter:

synapterAnalysis <- Synapter(l, master = TRUE)
fdr <- fpr <- 0.01
ppm <- 20

filterIdentPepScore(synapterAnalysis, method="BH", fdr=fdr)
filterQuantPepScore(synapterAnalysis, method="BH", fdr=fdr)
filterIdentProtFpr(synapterAnalysis, fpr=fpr)
filterQuantProtFpr(synapterAnalysis, fpr=fpr)
filterIdentPpmError(synapterAnalysis, ppm=ppm)
filterQuantPpmError(synapterAnalysis, ppm=ppm)
filterUniqueDbPeptides (synapterAnalysis, missedCleavages = 2, IisL = TRUE)

mergePeptides(synapterAnalysis)
modelRt(synapterAnalysis, span=0.1)
searchGrid(synapterAnalysis, 
           imdiffs = seq (0.6, 1.6, 0.2), ppm = 8:12,
           nsd = seq (1, 4, 1))

plotGrid (synapterAnalysis)
setBestGridParams (synapterAnalysis)
findEMRTs(synapterAnalysis)

fragmentMatching (synapterAnalysis)
fragmentMatchingStats <- plotFragmentMatchingPerformance (synapterAnalysis)$nonunique
sel <- which(fragmentMatchingStats[, "fdr"] < 0.05)
nonUniqueThreshold <- min (fragmentMatchingStats[sel, "deltacommon"])
cat (paste ("delta selected is", nonUniqueThreshold), sep = "\n")

filterUniqueMatches   (synapterAnalysis, minNumber = 1)
filterNonUniqueMatches(synapterAnalysis, minDelta = nonUniqueThreshold)

rescueEMRTs (synapterAnalysis, method = "rescue")

setwd (dirResult)
writeMatchedEMRTs  (synapterAnalysis, what = "full", file =  paste0 (outputFileNames[i], ".csv")) 

Now it seems that synapter struggles to put the result for filterNonUniqueMatches() in (leaving NA). See the jpeg attached

28 7 15 bug

PS: not sure whether it is related to rescue functionaility recent changes so opened it here

@sgibb
Copy link
Collaborator

sgibb commented Jul 28, 2015

Was not related to the recent changes. Was a regular bug 😉
matched.quant.spectrumIDs just contains the final match. Should it contain all matches?

@pavel-shliaha
Copy link
Collaborator Author

Yes, please

@sgibb
Copy link
Collaborator

sgibb commented Jul 28, 2015

Should be fixed now.

@sgibb sgibb closed this as completed Jul 28, 2015
@pavel-shliaha
Copy link
Collaborator Author

Sorry, Sebastian for not making myself clear,but there are problems again:

28 7 15 bug 2

@sgibb
Copy link
Collaborator

sgibb commented Jul 29, 2015

The plgsAgreement column is updated after findEMRTs, filterUniqueMatches and filterNonUniqueMatches automatically. Do you think we need a special user-visible function for it? (the NA you mentioned in your last comment was a result of the multiple ids in the matched.quant.spectrumIDs after my last fix of this issue; this is fixed now).

Also I reimport all Pep3D data.

@sgibb sgibb closed this as completed Jul 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants