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

Settings not opening on linux/arch based distro #99

Closed
sawdoctor opened this issue Sep 21, 2023 · 3 comments
Closed

Settings not opening on linux/arch based distro #99

sawdoctor opened this issue Sep 21, 2023 · 3 comments

Comments

@sawdoctor
Copy link

Hi there's an issue with the latest version on arch based distros where you can't open settings

Traceback (most recent call last):
File "Main.py", line 615, in OnToolClick
File "Main.py", line 1101, in _on_advanced_config
File "advanced_settings.py", line 219, in init
wx._core.wxAssertionError: C++ assertion ""Assert failure"" failed at /home/wxpy/wxPython-4.2.1/ext/wxWidgets/src/common/sizer.cpp(1601) in DoInsert(): too many items (19 > 2*9) in grid sizer (maybe you should omit the number of either rows or columns?)

I reported it to the aur maintainer and he confirmed he could reproduce the issue when building from source. Other than that the latest version seems to work fine on arch based distro's. When i saw the wxPython error I thought on no hahaha

@badabing2005
Copy link
Owner

Thanks for reporting.
I don't have access to arch to be able to test this.
If the aur maintainer can build it from source, can you ask him if this change fixes the issue?

Basically remove the rows=NUMROWS, part from this line.
image

Alternately this should also fix the issue by setting the NUMROWS to 11 on this line.
But I prefer the former if it works, this way I don't have to keep on updating the count each time I add widgets to the settings.
image

@MicMun
Copy link

MicMun commented Sep 23, 2023

I'm not the aur maintainer but I have a arch linux and tested it with building from source.
I tried the solution with removing rows=NUMROWS and build it. I can confirm, it works, I can open the settings with this fix.

@badabing2005
Copy link
Owner

OK thanks for confirming, I'll integrate this into the next release.

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

No branches or pull requests

3 participants