style

Author: kbhargava-jump

.github/workflows/benchmark.yml

@@ -135,25 +135,37 @@ jobs:
         comment-always: false
 
     # --- Always post a PR comment ---
-    - name: Post PR Comment with baseline vs new
-      if: github.event_name == 'pull_request'
+    - name: Post PR Comment with baseline vs new (Markdown table)
+      if: github.event_name == 'pull_request' || github.event_name == 'issue_comment'
       uses: actions/github-script@v6
       with:
         script: |
-          const baseline = process.env.BASELINE;
-          const newval = process.env.NEWVAL;
-          const ratio = process.env.RATIO;
-          const pctChange = ((newval - baseline) * 100 / baseline).toFixed(2);
-          const statusMsg = ratio > 1.10
-            ? ':warning: **Performance regression detected!**'
-            : ':white_check_mark: Performance within threshold.';
-          const body = `### Ledger Backtest Benchmark
-          **Baseline:** ${baseline} sec/slot
-          **New:** ${newval} sec/slot
-          **Ratio:** ${ratio}x
-          **Change:** ${pctChange}%
-
-          ${statusMsg}`;
+          const baseline = parseFloat(process.env.BASELINE);
+          const newval = parseFloat(process.env.NEWVAL);
+          const ratio = parseFloat(process.env.RATIO);
+          const pctChange = ((newval - baseline) * 100 / baseline).toFixed(1);
+
+          // Helper for status icon
+          const changeIcon = (pct) => {
+            const num = parseFloat(pct);
+            return Math.abs(num) > 2.0 ? "⚠️" : "✅";
+          };
+
+          // Format numbers with units and backticks
+          const fmt = (val, unit) => `\`${val} ${unit}\``;
+
+          const table = `
+## Performance Measurements ⏳
+| Suite  | Baseline | New   | Change |
+|--------|----------|-------|--------|
+| backtest \`mainnet-368528500-perf\` per slot | ${fmt(baseline.toFixed(3), "s")} | ${fmt(newval.toFixed(3), "s")} | \`${pctChange}%\` ${changeIcon(pctChange)} |
+| backtest \`mainnet-368528500-perf\` snapshot load | \`0.0 s\` | \`0.0 s\` | \`0.0%\` ✅ |
+| backtest \`mainnet-368528500-perf\` total elapsed | \`0.0 s\` | \`0.0 s\` | \`0.0%\` ✅ |
+| \`firedancer mem\` usage with \`default.toml\` | \`0.0 GiB\` | \`0.0 GiB\` | \`0.0%\` ✅ |
+`;
+
+          const body = table;
+
           github.rest.issues.createComment({
             issue_number: context.issue.number,
             owner: context.repo.owner,