fix: added cap to heatmapData

citizenfx · Jan 16, 2023 · 603f5aa · 603f5aa
1 parent 385dc4b
commit 603f5aa
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/core/components/StatsCollector/index.js b/core/components/StatsCollector/index.js
@@ -71,7 +71,7 @@ export default class StatsCollector {
                 const heatmapData = JSON.parse(rawFile);
                 if (!Array.isArray(heatmapData)) throw new Error('data is not an array');
                 if (!validatePerfCacheData(heatmapData)) throw new Error('invalid data in cache');
-                this.perfSeries = heatmapData;
+                this.perfSeries = heatmapData.slice(0, this.hardConfigs.performance.lengthCap);
             } catch (error) {
                 logError(`Failed to load stats_heatmapData_v1 with message: ${error.message}`);
                 logError('Since this is not a critical file, it will be reset.');
@@ -182,6 +182,9 @@ export default class StatsCollector {
 
         //Push to cache and save it
         this.perfSeries.push(currSnapshot);
+        if (this.perfSeries.length > this.hardConfigs.performance.lengthCap){
+            this.perfSeries.shift();
+        }
         try {
             await fse.outputJSON(this.hardConfigs.heatmapDataFile, this.perfSeries);
             if (verbose) {

diff --git a/docs/dev_notes.md b/docs/dev_notes.md
@@ -20,7 +20,7 @@
     - [x] guild roles
 - [x] bot: update AGAIN to djs v14
 - [x] fix `Restarting the fxserver with delay override 0.`
-- [ ] add cap to `stats_heatmapData_v1.json` (StatsCollector.hardConfigs.performance.lengthCap)
+- [x] add cap to `stats_heatmapData_v1.json` (StatsCollector.hardConfigs.performance.lengthCap)
 - [ ] Remove approved table from whitelist page?
 - [ ] the PR about hiding notifications
 - [ ] merge translations