Skip to content

Commit

Permalink
fix: add range to all Chroms, irrespective of type (in case we need i…
Browse files Browse the repository at this point in the history
…ndividual ones later), e.g. in TOPPView
  • Loading branch information
cbielow committed Nov 22, 2023
1 parent d54d28a commit 4a49009
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/openms/source/KERNEL/MSExperiment.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -287,8 +287,10 @@ namespace OpenMS
// update intensity, m/z and RT according to chromatograms as well:
for (ChromatogramType& cp : chromatograms_)
{
// update range of EACH chrom, if we need them individually later
cp.updateRanges();

// ignore TICs and ECs (as these are usually positioned at 0 and therefor lead to a large white margin in plots if included)
// ignore TICs and ECs for the whole experiments range (as these are usually positioned at 0 and therefor lead to a large white margin in plots if included)
if (cp.getChromatogramType() == ChromatogramSettings::TOTAL_ION_CURRENT_CHROMATOGRAM ||
cp.getChromatogramType() == ChromatogramSettings::EMISSION_CHROMATOGRAM)
{
Expand All @@ -299,7 +301,6 @@ namespace OpenMS

// ranges
this->extendMZ(cp.getMZ());// MZ
cp.updateRanges();
this->extend(cp);// RT and intensity from chroms's range
}
}
Expand Down

0 comments on commit 4a49009

Please sign in to comment.