Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

agnos 11.7 #34471

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

agnos 11.7 #34471

wants to merge 2 commits into from

Conversation

adeebshihadeh
Copy link
Contributor

@adeebshihadeh adeebshihadeh commented Jan 25, 2025

Pending approval from:

Release Checklist

  • test_onroad passes
  • Wi-Fi: lists networks and connects
  • Modem: connects to cell network
  • Image sizes haven't increased
  • Sounds work
  • Clean openpilot build: scons -c && scons -j8
  • Factory reset
    • from openpilot menu
    • tapping on boot
    • corrupt userdata
  • Color calibration
    • from /persist/comma/
    • directly from panel over sysfs
  • Clean setup: factory reset -> install openpilot -> openpilot works
  • AGNOS update works on warm boot
    • previous -> new
    • new -> previous
  • comma three: NVMe works

ABL

  • A/B slot fallback works
  • Boot time hasn't regressed (1s)

XBL

  • Display init works in cold and hot temperatures
  • Boot time hasn't regressed (2.4s)

Setup

  • (a) Not a real URL (e.g. comma, abc123, ...)
    • "Ensure the entered URL is valid"
    • Start over
    • Reboot device
  • (b) Website but not an installer URL (e.g. github.com, comma.ai, installer.comma.ai)
    • "No custom software found at this URL."
  • (c) Valid installer URL (e.g. openpilot.comma.ai)
    • Download successful (comma logo or installer appears)
    • /tmp/installer_url should contain the installer URL
    • Boots into openpilot

@incognitojam
Copy link
Member

flash can handle this with commaai/flash#87

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