gui not working on windows

[gvonkreisler]

File "d:\06_Python\iColoriT\iColoriT_demo\gui\gui_gamut.py", line 74, in paintEvent
painter.drawLine(x - w, y, x + w, y)
TypeError: arguments did not match any overloaded call: argument 1 has unexpected type 'float'

Install python 3.9 no its working

[RobBW]

I hit the same problem using MacOS Sonoma.
I use Warp as my CLI terminal and it has a very useful AI based natural language search feature which rapidlly identified the problem as integer functions being passed float values. It suggested the following corrections, changing 3 lines of code as follows:

iColoriT_demo\gui\gui_gamut.py", line 74, in paintEvent
painter.drawLine(x - w, y, x + w, y)
TypeError: arguments did not match any overloaded call: argument 1 has unexpected type 'float
The fix is to use int() in Lines 74 & 75:

painter.drawLine(int(x - w), int(y), int(x + w), int(y))
 painter.drawLine(int(x), int(y - w), int(x), int(y + w))

And iColoriT_demo/gui/ui_control.py", line 85, in update_painter
painter.drawRoundedRect(self.pnt.x() - w, self.pnt.y() - w, 1 + 2 * w, 1 + 2 * w, 2, 2)
Fix:
painter.drawRoundedRect(int(self.pnt.x() - w), int(self.pnt.y() - w), int(1 + 2 * w), int(1 + 2 * w), 2, 2

This seems to have fixed it

[YeolJ00]

@RobBW Hi, thanks for taking a look into the codes. I wasn't aware of these type checks that somehow didn't through an error in my environment. Your fix should do the work. Feel free to make a pull request or we will update the codes in the nearest availability.

Thanks again!

[RobBW]

I will not be able to test my suggested changes until I update to MacOS Sequoia. This will not happen until late December. I suggest that you generate the pull, I do not have the experience with github to be sure of getting it right Regards Rob ***@***.***

…

On 2 Dec 2024, at 19:03, JooYeol Yun ***@***.***> wrote: @RobBW <https://github.com/RobBW> Hi, thanks for taking a look into the codes. I wasn't aware of these type checks that somehow didn't through an error in my environment. Your fix should do the work. Feel free to make a pull request or we will update the codes in the nearest availability. Thanks again! — Reply to this email directly, view it on GitHub <#18 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABRNQJUQY7PIZBQ2IBX5O632DPZ4DAVCNFSM6AAAAABILQILLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJQGY2TCNRUGU>. You are receiving this because you were mentioned.