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

Add configuration UI #158

Merged
merged 59 commits into from
Apr 8, 2024
Merged

Conversation

lucasssvaz
Copy link
Contributor

@lucasssvaz lucasssvaz commented Feb 21, 2024

Arduino Static Libraries Configuration Editor

This is a simple application to configure the static libraries for the ESP32 Arduino core.
It allows the user to select the targets to compile, change the configuration options and compile the libraries.

The application is built using the 'textual' library, which is a Python library for building text-based user interfaces.

To Do

  • Design screens (main menu, editor, options, compilation)
  • Create events for all UI elements
  • Update from/to variables the UI elements
  • Properly pipe build script output to UI (currently freezes until subprocess finishes, even running in background task)
    • Fix target selection (bash script is ignoring target setting when executing from python script)
    • Make error and success messages easier to see in the compilation log
  • Change naming to make it less confusing (something like "IDF Configuration Options" and "Library Compilation Settings")
  • Separate main menu screen from app
    • Add key bindings to each screen
  • Change default Arduino folder to OS default
  • Make copying to Arduino optional
    • Dynamically change settings screen based on switch value
  • Add branch selection and debug level for IDF. Improve settings screen.
  • Add optional arguments to override default values
  • Make it pretty ✨ (any help is appreciated)
  • Write README

@lucasssvaz lucasssvaz marked this pull request as ready for review March 18, 2024 19:54
@lucasssvaz
Copy link
Contributor Author

lucasssvaz commented Apr 3, 2024

@SuGlider @P-R-O-C-H-Y Do you guys want to take a look at the UI or should we move on to the docker image implementation ?

@P-R-O-C-H-Y
Copy link
Member

@lucasssvaz I am on it :)

@P-R-O-C-H-Y
Copy link
Member

@lucasssvaz When #167 will get merged, it would be great to replace the Compilation target Selection with SelectionList, so multiple targets can be selected. Can you take a look? :)

@P-R-O-C-H-Y
Copy link
Member

Else the UI is really nice! Awesome job.

@lucasssvaz
Copy link
Contributor Author

lucasssvaz commented Apr 5, 2024

@me-no-dev @P-R-O-C-H-Y PTAL. I'll also add to the readme how to use it.

Copy link
Member

@P-R-O-C-H-Y P-R-O-C-H-Y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@me-no-dev me-no-dev merged commit 6026f5f into espressif:master Apr 8, 2024
8 checks passed
@lucasssvaz lucasssvaz deleted the feature/config_ui branch May 15, 2024 08:35
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.

3 participants