-
-
Notifications
You must be signed in to change notification settings - Fork 189
GUI + Installer #135
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
GUI + Installer #135
Conversation
requires tkinter
Ah, just realized I based this off the "renamed all instances of puterbot to openadapt" branch.. I guess that's okay for now. |
@abrichr replay quick demo (hardcoded but under 1 minute): EDIT: in-depth: ^ This is a good one. I teach through a recording, and have it replayed with a twist. At the end, I demo what visualization looks like. It's still a NaiveReplay but at least this one uses our API. view the branch here (gui+demo): https://github.com/0dm/OpenAdapt/tree/gui+demo |
added logo button
Excellent work @0dm ! For the gui, is there a reason you went with tk? What do you think about using NiceGUI with PyInstaller instead? |
I've used tkinter before so I wanted to give it a shot with this project. One benefit is that it comes installed with most python distributions, so less setup. |
Ok.. nicegui looks nice. I will try it out. |
code needs a little bit of clean up but is functional
could use it for exporting? or just sharing openadapt
todo: work on export
@0dm is this ready for re-review? |
Yes @abrichr |
from nicegui import ui | ||
|
||
|
||
class local_file_picker(ui.dialog): |
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.
Please use UpperCamelCase
convention for class names 🙏
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.
okay
openadapt/config.py
Outdated
"OPENAI_MODEL_NAME": "gpt-3.5-turbo", | ||
# may incur significant performance penalty | ||
"RECORD_READ_ACTIVE_ELEMENT_STATE": False, | ||
# TODO: remove? | ||
"REPLAY_STRIP_ELEMENT_STATE": True, | ||
"IGNORE_WARNINGS": True, |
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.
@0dm what do you think about setting this to False by default?
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.
I think it's better to ignore warnings by default if we use the GUI, since the logger will just fill up with warnings (until we add extra support for ObjC objects). Maybe I could have it set IGNORE_WARNINGS to true within the app though.
@0dm please fix merge conflicts 🙏 |
@abrichr resolved |
GUI + Installer
Resolves #134 (my own issue) - Created a GUI with tkinter
brew install python-tk@3.10
Contains:
Record Button with pop-up prompt for recording title:

Visualize Button
Replay Button
Clear Database Button
Log on the right side of the window (currently, Record does not work for this feature - keep getting SIGTRAP when I call record from the app, which just translates to an unhandled exception on MacOS
Please take a look, and let me know if this has any future. I didn't spend too long on it, so if it gets scrapped - no big deal, I actually find it quite useful for my own testing. The code may not be the most optimal, I'm not super good with tkinter but it gets the job done.
--
Instructions to run (latest - 2023/05/26):
python3 -m openadapt.app.main
To build:
python3 openadapt/app/build.py