[libcxx-commits] [libcxx] 42dd926 - [libc++] Switch back to plotting on revlist order for visualize-historical

[Louis Dionne via libcxx-commits]

Author: Louis Dionne
Date: 2025-09-26T07:47:21-04:00
New Revision: 42dd9260dbd615c075844c6c96ec6665be6c1aed

URL: https://github.com/llvm/llvm-project/commit/42dd9260dbd615c075844c6c96ec6665be6c1aed
DIFF: https://github.com/llvm/llvm-project/commit/42dd9260dbd615c075844c6c96ec6665be6c1aed.diff

LOG: [libc++] Switch back to plotting on revlist order for visualize-historical

That provides vastly better plots.

Added: 
    

Modified: 
    libcxx/utils/visualize-historical

Removed: 
    


################################################################################
diff  --git a/libcxx/utils/visualize-historical b/libcxx/utils/visualize-historical
index 00f52f09617bb..ef28e8b17ca4b 100755
--- a/libcxx/utils/visualize-historical
+++ b/libcxx/utils/visualize-historical
@@ -117,12 +117,12 @@ def create_plot(data, metric, subtitle=None):
     Create a plot object showing the evolution of each benchmark throughout the given commits for
     the given metric.
     """
-    data = data.sort_values(by=['date', 'benchmark'])
+    data = data.sort_values(by=['revlist_order', 'benchmark'])
     revlist = pandas.unique(data['commit']) # list of all commits in chronological order
     hover_info = {c: truncate_lines(c.show(), 30, marker='...').replace('\n', '<br>') for c in revlist}
     figure = plotly.express.scatter(data, title=f"{revlist[0].shortrev} to {revlist[-1].shortrev}",
                                           subtitle=subtitle,
-                                          x='date', y=metric,
+                                          x='revlist_order', y=metric,
                                           symbol='benchmark',
                                           color='benchmark',
                                           hover_name=[hover_info[c] for c in data['commit']],