Merge pull request #154 from faster-cpython/shorter-summary-tables

Don't include bases in the summary tables

Author: mdboom

Diff for: bench_runner/scripts/generate_results.py

@@ -245,13 +245,12 @@ def summarize_results(
 
     - The 3 most recent
     - Any results in the last 3 days
-    - The bases
     """
     results = list(results)
     new_results = []
     earliest = (datetime.date.today() - datetime.timedelta(days=days)).isoformat()
     for i, result in enumerate(results):
-        if i < n_recent or result.run_date >= earliest or result.version in bases:
+        if i < n_recent or result.run_date >= earliest:
             new_results.append(result)
     return new_results
 

Diff for: tests/test_generate_results.py

@@ -74,7 +74,7 @@ def _run_for_bases(bases, repo_path, force=False, has_base=[], check_readmes=Tru
 
     if check_readmes:
         contents = (repo_path / "README.md").read_text()
-        assert contents.count("\n|") == 7
+        assert contents.count("\n|") == 5
 
         contents = (repo_path / "RESULTS.md").read_text()
         assert contents.count("\n|") == 13