Ticket 54342: PanoramaWeb overwhelmed with requests for massive targetedms-groupChromatogramChart.view plots (#1155)

labkey-jeckels · web-flow · commit ac7e2a255418 · 2025-12-31T11:39:07.000-08:00
diff --git a/src/org/labkey/targetedms/chart/ChromatogramDataset.java b/src/org/labkey/targetedms/chart/ChromatogramDataset.java
@@ -1462,6 +1462,7 @@ protected String getSeriesLabel()
 
     public static class GroupDataset extends RtRangeDataset
     {
+        private static final int SERIES_LIMIT = 25;
         private final PeptideGroup _group;
         private final SampleFile _sampleFile;
         private final ViewContext _context;
@@ -1570,6 +1571,12 @@ protected List<PrecursorChromInfoPlus> getPrecursorChromInfos()
                 }
             }
 
+            // Ticket 54342: PanoramaWeb overwhelmed with requests for massive targetedms-groupChromatogramChart.view plots
+            if (_allMolecules.size() > SERIES_LIMIT)
+            {
+                _allMolecules = _allMolecules.entrySet().stream().limit(SERIES_LIMIT).collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue, (e1, e2) -> e1, LinkedHashMap::new));
+            }
+
             List<PrecursorChromInfoPlus> nonOptimizationPeaks = new ArrayList<>();
             for(List<PrecursorChromInfoPlus> infos : _allMolecules.values())
             {

Original file line number	Diff line number	Diff line change
`@@ -1462,6 +1462,7 @@ protected String getSeriesLabel()`
`1462`	`1462`
`1463`	`1463`	`public static class GroupDataset extends RtRangeDataset`
`1464`	`1464`	`{`
	`1465`	`+ private static final int SERIES_LIMIT = 25;`
`1465`	`1466`	`private final PeptideGroup _group;`
`1466`	`1467`	`private final SampleFile _sampleFile;`
`1467`	`1468`	`private final ViewContext _context;`
`@@ -1570,6 +1571,12 @@ protected List<PrecursorChromInfoPlus> getPrecursorChromInfos()`
`1570`	`1571`	`}`
`1571`	`1572`	`}`
`1572`	`1573`
	`1574`	`+ // Ticket 54342: PanoramaWeb overwhelmed with requests for massive targetedms-groupChromatogramChart.view plots`
	`1575`	`+ if (_allMolecules.size() > SERIES_LIMIT)`
	`1576`	`+ {`
	`1577`	`+ _allMolecules = _allMolecules.entrySet().stream().limit(SERIES_LIMIT).collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue, (e1, e2) -> e1, LinkedHashMap::new));`
	`1578`	`+ }`
	`1579`	`+`
`1573`	`1580`	`List<PrecursorChromInfoPlus> nonOptimizationPeaks = new ArrayList<>();`
`1574`	`1581`	`for(List<PrecursorChromInfoPlus> infos : _allMolecules.values())`
`1575`	`1582`	`{`