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

estimateNoise of Spectrum objects #78

Closed
lgatto opened this issue Feb 18, 2016 · 11 comments
Closed

estimateNoise of Spectrum objects #78

lgatto opened this issue Feb 18, 2016 · 11 comments

Comments

@lgatto
Copy link
Owner

lgatto commented Feb 18, 2016

@sgibb there would be some interest here to estimate signal-to-ratio using MSnbase. The MALDIquant::estimateNoise method seems to be a good candidate.

Have you ever tried it on MS2 spectra? Do the spectra need to be in profile mode, or is centroided fine too?

@sgibb
Copy link
Collaborator

sgibb commented Feb 21, 2016

MALDIquant::estimateNoise works on profile mode only. All noise estimators that I am aware of are metrics applied on fixed/moving windows (median/mad/quantile etc.), smoothing functions (loess, friedman' supersmoother, ...) or wavelet based methods. IHMO all these methods just work on profile data.
I am not sure whether it is possible to estimate noise in centroided MS2 spectra. What is the noise in these spectra (maybe there are some superfluous fragments?)?

@lgatto
Copy link
Owner Author

lgatto commented Feb 22, 2016

What is the noise in these spectra (maybe there are some superfluous fragments?)?

Yes, that's what the discussion centred around.

May be it would be worth adding the method for spectra that where centroided(x) == TRUE?

@sgibb
Copy link
Collaborator

sgibb commented Feb 22, 2016

On 2016-02-22 18:17:03, notifications@github.com wrote:

What is the noise in these spectra (maybe there are some superfluous fragments?)?

Yes, that's what the discussion centred around.

May be it would be worth adding the method for spectra that where centroided(x) == TRUE?
Did you mean centroided(x) == FALSE ?


Reply to this email directly or view it on GitHub:
#78 (comment)

@lgatto
Copy link
Owner Author

lgatto commented Feb 22, 2016

Did you mean centroided(x) == FALSE ?

Yes, indeed. The problem is that it is easy to fool this, as there is a centroided replacement method.

@lgatto
Copy link
Owner Author

lgatto commented Mar 9, 2016

I haven't heard of this request again. But it might be useful, at some point. to add these methods for Spectrum objects where centroided is FALSE.

@sgibb
Copy link
Collaborator

sgibb commented Mar 10, 2016

That would be no problem. The output of MALDIquant::estimateNoise is a matrix with two columns with the estimated noise per data point
(colnames(x) == c("mz", "intensity")). Would that be enough or do you want to
store the information somewhere in the Spectrum1 object?

On 2016-03-09 18:28:34, notifications@github.com wrote:

I haven't heard of this request again. But it might be useful, at some point. to add these methods for Spectrum objects where centroided is FALSE.


Reply to this email directly or view it on GitHub:
#78 (comment)

@lgatto
Copy link
Owner Author

lgatto commented Mar 10, 2016

There is not place to store this in Spectrum objects, so I think returning a matrix as in MALDIquant is the best solution.

@sgibb sgibb changed the title estimageNoise of Spectrum objects estimateNoise of Spectrum objects Mar 10, 2016
@sgibb sgibb closed this as completed in 9a05ad5 Mar 10, 2016
@lgatto
Copy link
Owner Author

lgatto commented Mar 10, 2016

Thank you. One question/comment - for empty spectra, your return NA. What would you think of returning a matrix of dimension 0 by 2 with appropriate column names, to make the output of estimateNoise consistent?

@sgibb
Copy link
Collaborator

sgibb commented Mar 10, 2016

Good point. Changed in 5a59ce1.
On 2016-03-10 19:45:58, notifications@github.com wrote:

Thank you. One question/comment - for empty spectra, your return NA. What would you think of returning a matrix of dimension 0 by 2 with appropriate column names, to make the output of estimateNoise consistent?


Reply to this email directly or view it on GitHub:
#78 (comment)

@lgatto
Copy link
Owner Author

lgatto commented Mar 11, 2016

I have alsoadded the estimateNoise,MSnExp method, that iterates over all spectra and returns a list of noise matrices 0038ebb.

@sgibb
Copy link
Collaborator

sgibb commented Mar 11, 2016

Great!
On 2016-03-11 10:48:47, notifications@github.com wrote:

I have alsoadded the estimateNoise,MSnExp method, that iterates over all spectra and returns a list of noise matrices 0038ebb.


Reply to this email directly or view it on GitHub:
#78 (comment)

lgatto pushed a commit that referenced this issue May 4, 2016
* master:
  add estimateNoise-MSnExp,method
  estimateNoise,MSnExp method
  import graphics, stats and utils in NAMESPACE
  estimateNoise returns an empty 2-column matrix for empty/centroided spectra
  add estimateNoise,Spectrum-method; closes #78
  new gh devel version

From: Laurent <lg390@cam.ac.uk>

git-svn-id: https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/MSnbase@114637 bc3139a8-67e5-0310-9ffc-ced21a209358
lgatto pushed a commit that referenced this issue Oct 27, 2016
* master:
  add estimateNoise-MSnExp,method
  estimateNoise,MSnExp method
  import graphics, stats and utils in NAMESPACE
  estimateNoise returns an empty 2-column matrix for empty/centroided spectra
  add estimateNoise,Spectrum-method; closes #78
  new gh devel version

From: Laurent <lg390@cam.ac.uk>

git-svn-id: https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/MSnbase@114637 bc3139a8-67e5-0310-9ffc-ced21a209358
lgatto pushed a commit that referenced this issue Sep 7, 2017
* master:
  add estimateNoise-MSnExp,method
  estimateNoise,MSnExp method
  import graphics, stats and utils in NAMESPACE
  estimateNoise returns an empty 2-column matrix for empty/centroided spectra
  add estimateNoise,Spectrum-method; closes #78
  new gh devel version

From: Laurent <lg390@cam.ac.uk>

git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/MSnbase@114637 bc3139a8-67e5-0310-9ffc-ced21a209358
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