Skip to content

Filter hacked submissions from leaderboard views#241

Merged
SinatrasC merged 1 commit into
gpu-mode:mainfrom
SinatrasC:filter-hacked-submissions-fresh
Jun 22, 2026
Merged

Filter hacked submissions from leaderboard views#241
SinatrasC merged 1 commit into
gpu-mode:mainfrom
SinatrasC:filter-hacked-submissions-fresh

Conversation

@SinatrasC

@SinatrasC SinatrasC commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exclude submissions marked hacked from leaderboard-derived views, summaries, trends, user search, and ranking notifications.
  • Check both leaderboard.submission.status and leaderboard.submission_job_status.status so existing and queued hacked markings are hidden from ranking surfaces.
  • Bump the ended-leaderboard top-users Redis cache prefix so stale cached rankings are not reused after deploy.
  • Add the local fixture schema needed by current leaderboard API tests and cover hacked filtering with a route-level regression test.

Why

  • Kernelbot can mark hacked submissions after the fact; kernelboard should stop showing those rows in public ranking surfaces without deleting submission history or run records.

Validation

  • ruff check kernelboard/ tests/
  • python3 -m py_compile kernelboard/api/leaderboard.py kernelboard/api/leaderboard_summaries.py kernelboard/index.py kernelboard/leaderboard.py ranking_worker.py tests/test_hacked_submission_filters.py tests/api/test_leaderboard_api.py
  • REDIS_URL=memory:// uv run --with-requirements requirements.txt pytest tests/api/test_leaderboard_api.py tests/api/test_leaderboard_summaries_api.py tests/test_hacked_submission_filters.py -q
  • git diff --check

@SinatrasC SinatrasC force-pushed the filter-hacked-submissions-fresh branch from 18acd26 to c3aa24f Compare June 21, 2026 22:15
@SinatrasC SinatrasC marked this pull request as ready for review June 21, 2026 22:17
@SinatrasC SinatrasC force-pushed the filter-hacked-submissions-fresh branch from c3aa24f to 255d10b Compare June 21, 2026 22:18
@SinatrasC SinatrasC merged commit a7406cd into gpu-mode:main Jun 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants