-
Notifications
You must be signed in to change notification settings - Fork 181
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
Launch fails with Python 3.10, Fix strict parameters for gui.move #599
Conversation
@@ -90,8 +90,8 @@ def main(): | |||
desktop = app.desktop() | |||
window_size = gui.size() | |||
gui.move( | |||
(desktop.width() - window_size.width()) / 2, | |||
(desktop.height() - window_size.height()) / 2, | |||
int((desktop.width() - window_size.width()) / 2), |
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.
You may want to do // 2
instead of casting float to int.
This PR solves #628 |
I'm not seeing the same thing @miquelmassot is. Doing the
and then a This is on Ubuntu 22.04 LTS. |
This remains an issue - please review this PR to see if this solves the problem. |
I created #644. It solves it for me. This PR would, too.
It does it. It is another issue that should be adressed in another PR. |
Please merge #642 to fix this. |
This bug fix from Ubuntu's packaging is likely to resolve this in at least 1 week before this is acted upon here. |
@micahflee can you take a look and merge this? |
Fixed in 8c69da1 |
Adding int() wrappers around the parameters in question fixed it.