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

QT_GUI ToggleSwitch Dtypes #7344

Closed
IsaiahHarvi opened this issue May 16, 2024 · 5 comments
Closed

QT_GUI ToggleSwitch Dtypes #7344

IsaiahHarvi opened this issue May 16, 2024 · 5 comments

Comments

@IsaiahHarvi
Copy link

What happened?

Turning on toggle switch to enable a python block sending data to virtual sink

System Information

OS: PopOS
GR Installation Method: apt

GNU Radio Version

3.10 (maint-3.10)

Specific Version

3.10.1.1

Steps to Reproduce the Problem

Running an embedded python block to a virtual sink

Relevant log output

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/gnuradio/qtgui/toggleswitch.py", line 143, in paintEvent
    painter.drawEllipse(
TypeError: arguments did not match any overloaded call:
  drawEllipse(self, r: QRectF): argument 1 has unexpected type 'float'
  drawEllipse(self, r: QRect): argument 1 has unexpected type 'float'
  drawEllipse(self, x: int, y: int, w: int, h: int): argument 1 has unexpected type 'float'
  drawEllipse(self, center: Union[QPointF, QPoint], rx: float, ry: float): argument 1 has unexpected type 'float'
  drawEllipse(self, center: QPoint, rx: int, ry: int): argument 1 has unexpected type 'float'
@IsaiahHarvi
Copy link
Author

IsaiahHarvi commented May 16, 2024

I was able to resolve this, locally, by type casting parameters to the drawEllipse calls inside of the paintEvent method

@marcusmueller
Copy link
Member

@IsaiahHarvi thanks for the report!

@marcusmueller
Copy link
Member

@marcusmueller
Copy link
Member

marcusmueller commented May 16, 2024

huh, interestingly, it works when I run it from the command line from which I'm also starting GRC-qt, but not from GRC-Qt through execute (the switch is just dead).

@marcusmueller
Copy link
Member

marcusmueller commented May 16, 2024

ah no wonder I can't reproduce – this was fixed by Volker Schroer in 3.10.2.0, in #5541 .

I don't know PopOS, but seeing it uses apt and it happens to have the old-ish variant still in Ubuntu 22.04LTS: probably, an operating system update will allow you to install a nwer version of GNU Radio, where this is already fixed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants