mx/Jungfraujoch
2
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 67 Wiki Activity

bragg integration: clarify that the trimmed background applies to stills too

Browse Source 
The background trimmed mean is gated on the beam (monochromatic vs broadband),
not on the acquisition mode: a monochromatic still (zero bandwidth) already
gets the trim by default, only broadband (non-zero bandwidth: pink-beam / DMM)
data keep the high-side sigma-clip. The comments, the settings doc and the
--background-trim help wrongly implied the trim was rotation-only / that stills
keep the sigma-clip. Wording only; no behaviour change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
leonarski_f
2026-07-17 09:23:13 +02:00
co-authored by Claude Opus 4.8
parent 7a9d25850c
commit 7d1fb8db77
3 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
image_analysis/bragg_integration/BraggIntegrationEngine.cpp
+3 -2
View File
@@ -68,8 +68,9 @@ BraggIntegrationEngine::BraggIntegrationEngine(const DiffractionExperiment &expe
beam_y = geom.GetBeamY_pxl();
use_ellipse = !empirical && (bw_sigma > 0.0 || c_radial > 0.0);
// Trimmed-mean background applies to monochromatic (rotation) data; broadband (stills) keep the tuned
// high-side sigma-clip, so the trim is forced off there. The fraction itself comes from the settings.
// Trimmed-mean background applies to monochromatic data - both rotation AND stills (the discriminator
// is the beam, not the acquisition mode). Broadband (non-zero bandwidth: pink-beam / DMM) data keep the
// tuned high-side sigma-clip instead, so the trim is forced off there. The fraction comes from settings.
bkg_trim = broadband ? 0.0f : settings.GetBackgroundTrimFraction();
polarization = experiment.GetPolarizationFactor();