Skip to content

Commit

Permalink
Remove baseline when estimating
Browse files Browse the repository at this point in the history
  • Loading branch information
lalo-caballero committed Dec 5, 2023
1 parent 30d8a40 commit 1c7412b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 2 additions & 2 deletions R/baseline-GCIMSDataset.R
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,12 @@
setMethod(
"estimateBaseline",
"GCIMSDataset",
function(object, dt_peak_fwhm_ms, dt_region_multiplier, rt_length_s) {
function(object, dt_peak_fwhm_ms, dt_region_multiplier, rt_length_s, remove =TRUE) {

delayed_op <- DelayedOperation(
name = "estimateBaseline",
fun = estimateBaseline,
params = list(dt_peak_fwhm_ms = dt_peak_fwhm_ms, dt_region_multiplier = dt_region_multiplier, rt_length_s = rt_length_s)
params = list(dt_peak_fwhm_ms = dt_peak_fwhm_ms, dt_region_multiplier = dt_region_multiplier, rt_length_s = rt_length_s, remove = remove)
)
object$appendDelayedOp(delayed_op)
# We recompute these, but maybe we could just reset them to zero...
Expand Down
5 changes: 4 additions & 1 deletion R/baseline-GCIMSSample.R
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
#' @export
methods::setMethod(
"estimateBaseline", "GCIMSSample",
function(object, dt_peak_fwhm_ms, dt_region_multiplier, rt_length_s) {
function(object, dt_peak_fwhm_ms, dt_region_multiplier, rt_length_s, remove) {
rt <- rtime(object)
dt <- dtime(object)
int <- intensity(object)
Expand All @@ -28,6 +28,9 @@ methods::setMethod(
rt_basel <- estimate_baseline_tr(int - dt_basel, region_size = rt_region_size_pts)
full_basel <- rt_basel + dt_basel
baseline(object) <- full_basel
if (remove){
object@data <- int - full_basel
}
object
}
)
Expand Down

0 comments on commit 1c7412b

Please sign in to comment.