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 an optional GUI using Gooey #4

Merged
merged 1 commit into from Dec 20, 2022
Merged

Add an optional GUI using Gooey #4

merged 1 commit into from Dec 20, 2022

Conversation

Ethkuil
Copy link
Contributor

@Ethkuil Ethkuil commented Dec 20, 2022

a simple gui

@Ethkuil Ethkuil changed the title Add a optional gui using Gooey Add an optional GUI using Gooey Dec 20, 2022
@fireattack
Copy link
Owner

fireattack commented Dec 20, 2022

Thanks!

Tested it, it works. I think it would be better if we use GooeyParser and add widget='FileChooser' to the original script, but that requires the user to install gooey, which is pretty heavy, even if they just want to use CLI.

I also have some trouble installing gooey due to two issues on wxPython:

  1. wxPython 4.2.0 builds with Python 3.11 but can't find wx._core wxWidgets/Phoenix#2297 -- the current build has issues on Py3.11, which is fixed but hasn't been released
  2. Installation with pip fails - cannot import name 'Mapping' from 'collections' wxWidgets/Phoenix#2236 or Pip install of wxPython 4.2.0 fails with python 3.11 wxWidgets/Phoenix#2296 -- users (in certain OS at least) need to manually install attrdict3 in order to be able to install wxPython

I solved them myself, but it would be confusing for others espeically newbies. So for now I will just merge it as-is.

@fireattack fireattack merged commit ed76dce into fireattack:master Dec 20, 2022
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.

None yet

2 participants