fix substr

emberjs · Apr 30, 2021 · 7bd2e41 · 7bd2e41
1 parent b1a033a
commit 7bd2e41
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/perf-check.yml b/.github/workflows/perf-check.yml
@@ -32,7 +32,7 @@ jobs:
           control-serve-command: yarn workspace relationship-performance-test-app ember s --path dist-control
           control-url: 'http://localhost:4200/materialization'
           markers: 'start-find-all,start-outer-materialization,stop-outer-materialization,end-find-all'
-          fidelity: high
+          fidelity: low
           upload-traces: true
           upload-results: true
       - name: Report TracerBench Results

diff --git a/bin/perf-tracking/create-comment.js b/bin/perf-tracking/create-comment.js
@@ -4,7 +4,7 @@ const path = require('path');
 const GITHUB_SHA = process.argv[2];
 
 const analysisPath = path.resolve(__dirname, '../../tracerbench-results/analysis-output.txt');
-let analysisText = fs.readFileSync(analysisPath);
+let analysisText = fs.readFileSync(analysisPath, 'utf-8');
 let lines = analysisText.substr(analysisText.indexOf('Benchmark Results Summary')).split('\n');
 
 lines = lines.filter((l) => l.indexOf('Phase') !== -1 || l.indexOf('phase') !== -1);