10 Oct
2021
10 Oct
'21
2:48 p.m.
Hi,
The commit 86b5fb30182c69ec38490346a4269d1ffce9ffb4 in gnuradio broke
osmocom_fft due to removing/renaming the pyqwidget wrapper.
This patch tries to fix that in a backwards-compatible manner.
From 645240939b50dc6f4cebd277c6a690a919f67fb5 Mon Sep
17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:31:23 +0200
Subject: [PATCH 1/3] follow pyqwidget rename in gnuradio
86b5fb30182c69ec38490346a4269d1ffce9ffb4
---
apps/osmocom_fft | 25 ++++++++++++++++++++-----
1 file changed, 20 insertions(+), 5 deletions(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index b23124a96f..6e072f8c6c 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -242,7 +242,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
from gnuradio import fosphor
self.scope = fosphor.qt_sink_c()
self.scope.set_frequency_range(0, input_rate)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(), Qt.QWidget)
self.scope_win.setMinimumSize(800, 300)
elif options.waterfall:
self.scope = qtgui.waterfall_sink_c(
@@ -256,7 +259,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
self.scope.enable_grid(False)
self.scope.enable_axis_labels(True)
self.scope.set_intensity_range(-100, 20)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(), Qt.QWidget)
self.scope_win.setMinimumSize(800, 420)
elif options.oscilloscope:
@@ -266,7 +272,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
name="",
nconnections=1
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(), Qt.QWidget)
self.scope_win.setMinimumSize(800, 600)
elif options.qtgui:
@@ -281,7 +290,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
plottime=True,
plotconst=True
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(), Qt.QWidget)
self.scope.set_update_time(1.0/10)
self.scope_win.setMinimumSize(800, 600)
@@ -294,7 +306,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
name="",
nconnections=1
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(), Qt.QWidget)
self.scope.disable_legend()
self.scope_win.setMinimumSize(800, 420)
--
2.30.2
CU,
Sec
--
Cooking without animal products is like doing sysadmin work without vi.
10 Oct
10 Oct
5:26 p.m.
Yes this was changed by https://github.com/gnuradio/gnuradio/pull/4994. We
did not know that function was being used externally.
On Sun, Oct 10, 2021 at 8:58 AM Stefan `Sec` Zehl <sec(a)42.org> wrote:
Hi,
The commit 86b5fb30182c69ec38490346a4269d1ffce9ffb4 in gnuradio broke
osmocom_fft due to removing/renaming the pyqwidget wrapper.
This patch tries to fix that in a backwards-compatible manner.
From 645240939b50dc6f4cebd277c6a690a919f67fb5 Mon Sep 17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:31:23 +0200
Subject: [PATCH 1/3] follow pyqwidget rename in gnuradio
86b5fb30182c69ec38490346a4269d1ffce9ffb4
---
apps/osmocom_fft | 25 ++++++++++++++++++++-----
1 file changed, 20 insertions(+), 5 deletions(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index b23124a96f..6e072f8c6c 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -242,7 +242,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
from gnuradio import fosphor
self.scope = fosphor.qt_sink_c()
self.scope.set_frequency_range(0, input_rate)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(),
Qt.QWidget)
self.scope_win.setMinimumSize(800, 300)
elif options.waterfall:
self.scope = qtgui.waterfall_sink_c(
@@ -256,7 +259,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
self.scope.enable_grid(False)
self.scope.enable_axis_labels(True)
self.scope.set_intensity_range(-100, 20)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(),
Qt.QWidget)
self.scope_win.setMinimumSize(800, 420)
elif options.oscilloscope:
@@ -266,7 +272,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
name="",
nconnections=1
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(),
Qt.QWidget)
self.scope_win.setMinimumSize(800, 600)
elif options.qtgui:
@@ -281,7 +290,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
plottime=True,
plotconst=True
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(),
Qt.QWidget)
self.scope.set_update_time(1.0/10)
self.scope_win.setMinimumSize(800, 600)
@@ -294,7 +306,10 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
name="",
nconnections=1
)
- self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ if hasattr(self.scope,'pyqwidget'):
+ self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
+ else:
+ self.scope_win = sip.wrapinstance(self.scope.qwidget(),
Qt.QWidget)
self.scope.disable_legend()
self.scope_win.setMinimumSize(800, 420)
--
2.30.2
CU,
Sec
--
Cooking without animal products is like doing sysadmin work without vi.
5:53 p.m.
While playing with osmocom_fft, i found two additional issues:
- the '-S' option fails due to a RuntimeError being changed to a ValueError
- the '-Q' option fails due to a missing "parent" parameter.
the first patch should be backwards-compatible.
The second one i've only tested with gnuradio v3.9.3.0 / maint-3.9
From 4411edd838236257e00825717a819989528e2cdb Mon Sep
17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:32:50 +0200
Subject: [PATCH 2/3] fix oscilloscope display
missing message port now returns ValueError
---
apps/osmocom_fft | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index 6e072f8c6c..40cba8cc14 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -317,7 +317,7 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
try:
self.freq = freq_recv(self.set_freq)
self.msg_connect((self.scope, 'freq'), (self.freq, 'msg'))
- except RuntimeError:
+ except (RuntimeError, ValueError):
self.freq = None
self.file_sink = blocks.file_sink(gr.sizeof_gr_complex, "/dev/null",
False)
--
2.30.2
From c8b2bfd337adcfe1c1489ba9ec44fd56833d57e4 Mon Sep
17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:34:09 +0200
Subject: [PATCH 3/3] fix QTgui 'all-in-one' display
pybind seems to require the parent parameter now.
---
apps/osmocom_fft | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index 40cba8cc14..f829192cd6 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -288,7 +288,8 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
plotfreq=True,
plotwaterfall=True,
plottime=True,
- plotconst=True
+ plotconst=True,
+ parent=None
)
if hasattr(self.scope,'pyqwidget'):
self.scope_win = sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
--
2.30.2
11 Oct
11 Oct
1:04 p.m.
We'll fix the pyqwidget() regression in GNU Radio, and I'm going to put out
another v3.9 release soon to address this. So, no need to fix on this end.
Do you think the second set of patches relates to something that changed
during 3.9? Those look like they might be 3.8 vs 3.9. Was osmocom_fft
previously working under 3.9?
On Sun, Oct 10, 2021 at 11:54 AM Stefan `Sec` Zehl <sec(a)42.org> wrote:
While playing with osmocom_fft, i found two additional
issues:
- the '-S' option fails due to a RuntimeError being changed to a ValueError
- the '-Q' option fails due to a missing "parent" parameter.
the first patch should be backwards-compatible.
The second one i've only tested with gnuradio v3.9.3.0 / maint-3.9
From 4411edd838236257e00825717a819989528e2cdb Mon Sep 17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:32:50 +0200
Subject: [PATCH 2/3] fix oscilloscope display
missing message port now returns ValueError
---
apps/osmocom_fft | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index 6e072f8c6c..40cba8cc14 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -317,7 +317,7 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
try:
self.freq = freq_recv(self.set_freq)
self.msg_connect((self.scope, 'freq'), (self.freq, 'msg'))
- except RuntimeError:
+ except (RuntimeError, ValueError):
self.freq = None
self.file_sink = blocks.file_sink(gr.sizeof_gr_complex,
"/dev/null", False)
--
2.30.2
From c8b2bfd337adcfe1c1489ba9ec44fd56833d57e4 Mon Sep 17 00:00:00 2001
From: Stefan `Sec` Zehl <sec(a)42.org>
Date: Sun, 10 Oct 2021 14:34:09 +0200
Subject: [PATCH 3/3] fix QTgui 'all-in-one' display
pybind seems to require the parent parameter now.
---
apps/osmocom_fft | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/apps/osmocom_fft b/apps/osmocom_fft
index 40cba8cc14..f829192cd6 100755
--- a/apps/osmocom_fft
+++ b/apps/osmocom_fft
@@ -288,7 +288,8 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
plotfreq=True,
plotwaterfall=True,
plottime=True,
- plotconst=True
+ plotconst=True,
+ parent=None
)
if hasattr(self.scope,'pyqwidget'):
self.scope_win = sip.wrapinstance(self.scope.pyqwidget(),
Qt.QWidget)
--
2.30.2
3:35 p.m.
GNU Radio maint-3.9 branch has been updated with this fix.
On Mon, Oct 11, 2021 at 7:04 AM Jeff Long <willcode4(a)gmail.com> wrote:
We'll fix the pyqwidget() regression in GNU Radio,
and I'm going to put
out another v3.9 release soon to address this. So, no need to fix on this
end. Do you think the second set of patches relates to something that
changed during 3.9? Those look like they might be 3.8 vs 3.9. Was
osmocom_fft previously working under 3.9?
On Sun, Oct 10, 2021 at 11:54 AM Stefan `Sec` Zehl <sec(a)42.org> wrote:
> While playing with osmocom_fft, i found two additional issues:
>
> - the '-S' option fails due to a RuntimeError being changed to a
> ValueError
> - the '-Q' option fails due to a missing "parent" parameter.
>
> the first patch should be backwards-compatible.
> The second one i've only tested with gnuradio v3.9.3.0 / maint-3.9
>
>
> From 4411edd838236257e00825717a819989528e2cdb Mon Sep 17 00:00:00 2001
> From: Stefan `Sec` Zehl <sec(a)42.org>
> Date: Sun, 10 Oct 2021 14:32:50 +0200
> Subject: [PATCH 2/3] fix oscilloscope display
>
> missing message port now returns ValueError
> ---
> apps/osmocom_fft | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/apps/osmocom_fft b/apps/osmocom_fft
> index 6e072f8c6c..40cba8cc14 100755
> --- a/apps/osmocom_fft
> +++ b/apps/osmocom_fft
> @@ -317,7 +317,7 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
> try:
> self.freq = freq_recv(self.set_freq)
> self.msg_connect((self.scope, 'freq'), (self.freq,
'msg'))
> - except RuntimeError:
> + except (RuntimeError, ValueError):
> self.freq = None
>
> self.file_sink = blocks.file_sink(gr.sizeof_gr_complex,
> "/dev/null", False)
> --
> 2.30.2
>
> From c8b2bfd337adcfe1c1489ba9ec44fd56833d57e4 Mon Sep 17 00:00:00 2001
> From: Stefan `Sec` Zehl <sec(a)42.org>
> Date: Sun, 10 Oct 2021 14:34:09 +0200
> Subject: [PATCH 3/3] fix QTgui 'all-in-one' display
>
> pybind seems to require the parent parameter now.
> ---
> apps/osmocom_fft | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/apps/osmocom_fft b/apps/osmocom_fft
> index 40cba8cc14..f829192cd6 100755
> --- a/apps/osmocom_fft
> +++ b/apps/osmocom_fft
> @@ -288,7 +288,8 @@ class app_top_block(gr.top_block, Qt.QMainWindow):
> plotfreq=True,
> plotwaterfall=True,
> plottime=True,
> - plotconst=True
> + plotconst=True,
> + parent=None
> )
> if hasattr(self.scope,'pyqwidget'):
> self.scope_win =
> sip.wrapinstance(self.scope.pyqwidget(), Qt.QWidget)
> --
> 2.30.2
>
>
>
>
16 Oct
16 Oct
6:51 p.m.
Hi,
On Mon, Oct 11, 2021 at 07:04 -0400, Jeff Long wrote:
Do you think the second set of patches relates to
something that changed
during 3.9? Those look like they might be 3.8 vs 3.9. Was osmocom_fft
previously working under 3.9?
I'm pretty sure it worked back when I ported osmocom_fft to 3.9 in Feb
of 2020. I have no idea when it started breaking as I don't use the -S /
-Q optiones usually.
CU,
Sec
--
Error reduces
Your expensive computer
To a simple stone
931
days inactive
937
days old
5 comments
2 participants
participants (2)
-
Jeff Long -
Stefan `Sec` Zehl