-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
UI update #1173
UI update #1173
Conversation
- Allowing multiple modifiers to the config in train screen - Allow saving overwritten config variables into the myconfig.py file - Add window to show all training config parameters of trained pilots in the train screen - Allow overwriting training comments in the train screen - Allow selection of columns in the pilot panel in the train screen - Allowing multiple pilots in the pilot screen - Integrating tubplot into the pilot screen - Switching the tub graph to native matplotlib in tub screen - Refactor UI into multiple files - Improve UI layout using dark rounded backgrounds for grouping widgets for each functionality Improvement to image caching: - Replace binary flag CACHE_IMAGES by 3 flags for no-caching, caching as np uint8 arrays or caching as jpeg binaries which is a good alternative if caching as uint8 images uses too much memory, but binary jpegs do still fit.
- bugfix for missing processor in image() method of TubRecord Fix pytorch packages and model
# Conflicts: # donkeycar/__init__.py # donkeycar/parts/pytorch/torch_train.py # donkeycar/tests/test_torch.py # setup.cfg
…dependencies pull a very old version that doesn't work with tensorflow.
…s from other modules are printed and over-flooding the shell.
You would write: |
thanks |
This is what I got on a fresh RPi OS 64bit installed with RPi image on a RPi 5
|
I uninstalled 5.2.dev3 and installed 5.2.dev3 and I saw it installing kivy, matplotlib, etc. Then donkey ui launched correctly. |
There is a problem with the car connector in the case where we are running donkey ui from the car. I tried using |
It does not make sense to connect from the RPi through |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. The refactor of the ui code is really good. I'm sorry it so long to get to this. Thank you for your patience and for this great work.
Summary
UI changes
The UI code has been in a single python and single kv file which is hard to maintain. Also, the kivy version 2.0 is by now quite outdated. The UI was crashing on RPi. Here is a refactoring that resolves these issues and adds a lot more features in addition:
donkey tubplot
has been removed as it was not working anyway.sudo apt install libhdf5-dev
before.Other changes
We have introduced a more versatile image caching policy to support a binary caching format in addition to the no cache and the numpy array which we have now. The binary format stores the jpeg compression which gives a good compromise between performance and memory footprint. Training with 100k records and binary caching using 32GB or ram w/o swapping is possible.
To install this code
I pushed this code to test-pypi and it can be installed with
pip install -i https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple donkeycar-pypi\[XX\]==5.2.dev3
, whereXX
is eitherpc
orpi
ornano
. No escaping of[
or]
needed if you are on bash.Screenshots
Tub manager
Trainer
Pilot arena
Car connector