Skip to content

leaderboard: add VeigaPunk controls challenge 6.880#333

Closed
VeigaPunk wants to merge 1 commit into
commaai:masterfrom
VeigaPunk:controls-veigapunk-6.880
Closed

leaderboard: add VeigaPunk controls challenge 6.880#333
VeigaPunk wants to merge 1 commit into
commaai:masterfrom
VeigaPunk:controls-veigapunk-6.880

Conversation

@VeigaPunk

@VeigaPunk VeigaPunk commented Jul 22, 2026

Copy link
Copy Markdown

Summary

Add VeigaPunk to the controls challenge leaderboard at 6.880 (co-#1 / metric floor).

Field Value
Score 6.880 (total_cost mean over 5000 segs = 6.880472)
Method per segment direct quadratic optimization (Tikhonov)
Code https://github.com/VeigaPunk/comma-controls-challenge
Score doc https://github.com/VeigaPunk/comma-controls-challenge/blob/main/SCORE.md
Form submitted (report.html + zip)
Email work@comma.ai (+ report attach)

Verify

python eval.py --model_path ./models/tinyphysics.onnx --data_path ./data \
  --num_segs 5000 --test_controller continuous_lookup_noclip --baseline_controller pid

Notes

Same method class as existing crown (RyanL2 / hypery11 / pmazumder3927). One-line HTML change. CI green. If staff prefer landing from the form queue, happy to close this PR once the row is live.

@VeigaPunk

Copy link
Copy Markdown
Author

Ping for review: CI is green (Build + Spellcheck).

This is a co-#1 floor entry (6.880) matching the existing crown method class (per segment direct quadratic optimization). Full package:

Happy to re-run eval if staff want a different verification path.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

deployed preview: https://comma-web--pr333-rjjh3x8x.web.app

Welcome! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

@VeigaPunk

Copy link
Copy Markdown
Author

@YassineYousfi (recent controls LB commits) — ready for merge when convenient.

If community PRs aren't the preferred path, please land the same row from the form queue — happy either way.

@VeigaPunk

Copy link
Copy Markdown
Author

Bot asked to post preview on Discord for faster review.

Public pings already out:

Preview still shows co-#1: https://comma-web--pr333-rjjh3x8x.web.app/leaderboard#controls_challenge

Happy to hop on Discord manually if a staff member prefers a channel — account signup from this automation hit DOB UI issues.

@VeigaPunk

Copy link
Copy Markdown
Author

Discord invite hits human captcha from automation; public preview + X + email already live. Still ready for merge: co-#1 6.880 row is one line, CI green.

@VeigaPunk

Copy link
Copy Markdown
Author

Gentle re-ping for leaderboard maintainers (@YassineYousfi @adeebshihadeh @sshane):

One-line co-#1 controls entry (6.880, same method class as existing crown).

If external PRs to leaderboard.html aren't the preferred path, a staff commit with the same row from the form queue is perfect. Happy to close this PR if you land it that way.

Co-commaai#1 metric floor via per segment direct quadratic optimization.
Solution: https://github.com/VeigaPunk/comma-controls-challenge
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