mirror of
https://github.com/sunnypilot/sunnypilot.git
synced 2026-08-23 04:33:48 +08:00
91b7752268
* pressed state for larger sliders
* wifibutton
* fix
* clean up
* some work
* don't nee this now
* stash
* more
* new pressed bigcircle
* black
* interp
* just check position
* clean up and fix slider reset
* fix custom
* no speed
* stash
* even chatter couldn't figure this one out
* makes sense to combine together, less split mentality
* clean that up
* fix lag
* match ui.py prio to eliminate lag on wifiui show event. separately, why is this slow?
* night mode
* delay scroll over
* fix auto scrolling
* stash
* waiting looks disabled
* clean up and don't reset sliders until user goes back
* rm
* fix
* add termsheader back
* fix callbacks
* ctrl alt l
* fix text spacing
* clean up
* stash
* fix style
* i want to go back
* guard on exit
* kinda useless stuff
* Revert "kinda useless stuff"
This reverts commit a4acbac31523408f358c5f68262cb630aa13ad8e.
* Revert "guard on exit"
This reverts commit 63ccfbf64edfbe1a144a441681f5ec78d8021ff7.
* wide
* setup pressed!
* grow animation
* 10s after initial
* slow fast
* start onboarding (terms)
* rm duplicate page
* add qr code
* final grey
* fix visual lag on first start
* clean up dead code
* dont exit from cancel
* revert grey
* clean up, REVIEW ME
* Revert "clean up, REVIEW ME"
This reverts commit c66fa60947c5f922520e7cf58c630b4bbe2d0177.
* reboot slider
* kb fix
* Revert "kb fix"
This reverts commit 883039448e6c37ae1d25d4f75ada6e96b6736358.
* ./ goes to letters
* Revert "./ goes to letters"
This reverts commit 0d97442427edb1a000638863a3f2181204ddc160.
* clean up
* some more clean up
* more
* clean up
* rename block
* reset pending scroll so it can't use stale data in rare sequence
* remove unused assets
* clean up imports
* fix updater
* clean up
* fix double reboot
* demo time - reset to setup on reboot
* let manager restart
* Revert "demo time - reset to setup on reboot"
This reverts commit 9468657e8438a1ce8fcb5266403b7bb3539f131f.
* url... and no grow animation on start button
* one next button
* grow instead of shake wifi button
* 36 pt font size in setup
* touch up onboarding a lil
* Revert "rm cpp bz2 (#37332)"
This reverts commit f4a36f7f74.
* more onboarding and clean up
* clean up
* wow what an amazing future clean up
* back to software select
* fix
* copy
* fix dm confirmation dialog not disabling widget underneath, all fixed with real nav stack in here
* uploading
* lint
* add review terms to device w/ close button
* todo
* remove old Terms vertical scrolling classes
* use new Scroller!
* installer
* tweak to match figma exactly
* revert
* fixup updater
* demo day
* demo day v2
* ... for percent while finishing setup
* demo day v3
* demo day v4
* remove ...
* demo day v6 -- "why does it do that!!"
* demo day v7 -- no flash
* hmm
* demo day v7
* prebuilt
* revert demo day
* scroll after pop animation
* back -> retry
* stash fixes
* damn, need back_callback
* scroll over immediately if already in network setup
* tweaks
* going down is confusing
* more
* Revert "more"
This reverts commit 29ce75b1f81eb40e7527a71d27842d9a66802206.
* Revert "going down is confusing"
This reverts commit 0cd2ae30d4135db1ccba6478429b45e886714e9c.
* dupl
* nl
* sort functions
* more clean up from merge
* move
* more
* dismiss to download (hack)
* Revert "dismiss to download (hack)"
This reverts commit 53c45ed1f63db1f0cebbce0dfab1777c8658f505.
* onboarding work
* set brightness and timeout in root onboarding only
* clean up
* type
* keep 5m for settings preview
* switch back to letters on . or /
* reset first step scroller
* custom software warning goes down network comes up and back cb fix
* clean up
* smaller qr
* ReviewTermsPage just for device as NavWidget
* clean up
* installer: stay on 100%
* reset has internet while in wifiui
* try this
* try this
* see what error we get exactly
see what error we get exactly
* not final solution but see how good
* rm
* copy changes
* reset on disconnect
* for separate pr
* Revert "reset on disconnect"
This reverts commit 552372fa4d497ba7d9de7f2edb730ee63798ffa4.
* revert this, too buggy
* fix for updater
* sort
* fix test
* minor cleanup
* more leaks than this rn
* onboarding clean up
* clean up application
* click delay to small button
* clean up
* reset more state
* fix training guide not cleaning up driverview
* Revert "fix training guide not cleaning up driverview"
This reverts commit cac7c5f436056cc9e747f80905d390790fb83c22.
* simpler fix :(
* nice catch, if you go back to terms it will reset 300s timeout and brightness
* duplicate show
* unused
120 lines
4.9 KiB
Python
Executable File
120 lines
4.9 KiB
Python
Executable File
import pyray as rl
|
|
rl.set_config_flags(rl.ConfigFlags.FLAG_WINDOW_HIDDEN)
|
|
import gc
|
|
import weakref
|
|
import pytest
|
|
from openpilot.system.ui.lib.application import gui_app
|
|
from openpilot.system.ui.widgets import Widget
|
|
|
|
# mici dialogs
|
|
from openpilot.selfdrive.ui.mici.layouts.onboarding import TrainingGuide as MiciTrainingGuide, OnboardingWindow as MiciOnboardingWindow
|
|
from openpilot.selfdrive.ui.mici.onroad.driver_camera_dialog import DriverCameraDialog as MiciDriverCameraDialog
|
|
from openpilot.selfdrive.ui.mici.widgets.pairing_dialog import PairingDialog as MiciPairingDialog
|
|
from openpilot.selfdrive.ui.mici.widgets.dialog import BigDialog, BigConfirmationDialogV2, BigInputDialog
|
|
from openpilot.selfdrive.ui.mici.layouts.settings.device import MiciFccModal
|
|
|
|
# tici dialogs
|
|
from openpilot.selfdrive.ui.onroad.driver_camera_dialog import DriverCameraDialog as TiciDriverCameraDialog
|
|
from openpilot.selfdrive.ui.layouts.onboarding import OnboardingWindow as TiciOnboardingWindow
|
|
from openpilot.selfdrive.ui.widgets.pairing_dialog import PairingDialog as TiciPairingDialog
|
|
from openpilot.system.ui.widgets.confirm_dialog import ConfirmDialog
|
|
from openpilot.system.ui.widgets.option_dialog import MultiOptionDialog
|
|
from openpilot.system.ui.widgets.html_render import HtmlModal
|
|
from openpilot.system.ui.widgets.keyboard import Keyboard
|
|
|
|
# FIXME: known small leaks not worth worrying about at the moment
|
|
KNOWN_LEAKS = {
|
|
"openpilot.selfdrive.ui.mici.onroad.driver_camera_dialog.DriverCameraView",
|
|
"openpilot.selfdrive.ui.mici.layouts.onboarding.TermsPage",
|
|
"openpilot.selfdrive.ui.mici.layouts.onboarding.TrainingGuide",
|
|
"openpilot.selfdrive.ui.mici.layouts.onboarding.DeclinePage",
|
|
"openpilot.selfdrive.ui.mici.layouts.onboarding.OnboardingWindow",
|
|
"openpilot.selfdrive.ui.onroad.driver_state.DriverStateRenderer",
|
|
"openpilot.selfdrive.ui.onroad.driver_camera_dialog.DriverCameraDialog",
|
|
"openpilot.selfdrive.ui.layouts.onboarding.TermsPage",
|
|
"openpilot.selfdrive.ui.layouts.onboarding.DeclinePage",
|
|
"openpilot.selfdrive.ui.layouts.onboarding.OnboardingWindow",
|
|
"openpilot.system.ui.widgets.confirm_dialog.ConfirmDialog",
|
|
"openpilot.system.ui.widgets.label.Label",
|
|
"openpilot.system.ui.widgets.button.Button",
|
|
"openpilot.system.ui.widgets.html_render.HtmlRenderer",
|
|
"openpilot.system.ui.widgets.nav_widget.NavBar",
|
|
"openpilot.selfdrive.ui.mici.layouts.settings.device.MiciFccModal",
|
|
"openpilot.system.ui.widgets.inputbox.InputBox",
|
|
"openpilot.system.ui.widgets.scroller_tici.Scroller",
|
|
"openpilot.system.ui.widgets.label.UnifiedLabel",
|
|
"openpilot.system.ui.widgets.mici_keyboard.MiciKeyboard",
|
|
"openpilot.selfdrive.ui.mici.widgets.dialog.BigConfirmationDialogV2",
|
|
"openpilot.system.ui.widgets.keyboard.Keyboard",
|
|
"openpilot.system.ui.widgets.slider.BigSlider",
|
|
"openpilot.selfdrive.ui.mici.widgets.dialog.BigInputDialog",
|
|
"openpilot.system.ui.widgets.option_dialog.MultiOptionDialog",
|
|
}
|
|
|
|
|
|
def get_child_widgets(widget: Widget) -> list[Widget]:
|
|
children = []
|
|
for val in widget.__dict__.values():
|
|
items = val if isinstance(val, (list, tuple)) else (val,)
|
|
children.extend(w for w in items if isinstance(w, Widget))
|
|
return children
|
|
|
|
|
|
@pytest.mark.skip(reason="segfaults")
|
|
def test_dialogs_do_not_leak():
|
|
gui_app.init_window("ref-test")
|
|
|
|
leaked_widgets = set()
|
|
|
|
for ctor in (
|
|
# mici
|
|
MiciDriverCameraDialog, MiciPairingDialog,
|
|
lambda: MiciTrainingGuide(lambda: None),
|
|
lambda: MiciOnboardingWindow(lambda: None),
|
|
lambda: BigDialog("test", "test"),
|
|
lambda: BigConfirmationDialogV2("test", "icons_mici/settings/network/new/trash.png"),
|
|
lambda: BigInputDialog("test"),
|
|
lambda: MiciFccModal(text="test"),
|
|
# tici
|
|
TiciDriverCameraDialog, TiciOnboardingWindow, TiciPairingDialog, Keyboard,
|
|
lambda: ConfirmDialog("test", "ok"),
|
|
lambda: MultiOptionDialog("test", ["a", "b"]),
|
|
lambda: HtmlModal(text="test"),
|
|
):
|
|
widget = ctor()
|
|
all_refs = [weakref.ref(w) for w in get_child_widgets(widget) + [widget]]
|
|
|
|
del widget
|
|
|
|
for ref in all_refs:
|
|
if ref() is not None:
|
|
obj = ref()
|
|
name = f"{type(obj).__module__}.{type(obj).__qualname__}"
|
|
leaked_widgets.add(name)
|
|
|
|
print(f"\n=== Widget {name} alive after del")
|
|
print(" Referrers:")
|
|
for r in gc.get_referrers(obj):
|
|
if r is obj:
|
|
continue
|
|
|
|
if hasattr(r, '__self__') and r.__self__ is not obj:
|
|
print(f" bound method: {type(r.__self__).__qualname__}.{r.__name__}")
|
|
elif hasattr(r, '__func__'):
|
|
print(f" method: {r.__name__}")
|
|
else:
|
|
print(f" {type(r).__module__}.{type(r).__qualname__}")
|
|
del obj
|
|
|
|
gui_app.close()
|
|
|
|
unexpected = leaked_widgets - KNOWN_LEAKS
|
|
assert not unexpected, f"New leaked widgets: {unexpected}"
|
|
|
|
fixed = KNOWN_LEAKS - leaked_widgets
|
|
assert not fixed, f"These leaks are fixed, remove from KNOWN_LEAKS: {fixed}"
|
|
|
|
|
|
if __name__ == "__main__":
|
|
test_dialogs_do_not_leak()
|