-
Notifications
You must be signed in to change notification settings - Fork 10
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
gaussNorm: I don't understand the resulting transformation #54
Comments
@SamGG, just a heads-up that the authors of the gaussNorm approach are from this paper |
Hi Greg. Thanks for your quick feedback. Except Ryan, I don't see these names nowdays, and he seems not to be on github. May @tslumley or @fhahne answer? My main point was to warn potential users and to keep a track if someone wants to investigate. What I coded for 2 peaks is:
This leads to correct results IMHO when s is the same for the 2 peaks. s = c(0.5, 0.5) s = c(0.25, 0.25) s = c(0.25, 0.5) The latter is still slightly weird (although comprehensive) as s is not similar. This should not arise as the API does not provide to adjust it. BTW, the estimation of s is the sd of all the intensity of the channel (code below). This relies on all the data, not of each peak, which is a coarse estimation in my view. Lines 472 to 475 in fabb131
|
@SamGG I would need to see more practical examples that show the data -- not just the diff -- in order to understand more clearly, though I don't doubt you've found some sensible improvement. However, separately, I also think this algorithm has other noteworthy problems so I'm not sure how much thought I can give this particular one 😁 If I recall correctly:
The core of the warping, when it gets it right, seems pretty good. Unfortunately there are many situations where it fails. If the core warping strategy were wrapped with a more robust surrounding layer, that could be valuable. |
I see problems with this code:
flowStats/R/gaussNorm.R
Lines 497 to 500 in fabb131
Code to reproduce my trials
Here are the 2nd and 4th graphs showing the shift applied to the data as a function of the intensity (from asinh(FluoIntensity/cofactor)).
@mikejiang @SofieVG @cciccole
The text was updated successfully, but these errors were encountered: