Forum: GNU Radio Bug: gr-qtgui: wrong design of variable-type blocks for GRC (category /GUI Widgets/QT)

F7b45a110fba31e5b37c60978bc8d3b7?d=identicon&s=25 Artem Pisarenko (Guest)
on 2013-12-24 09:40
(Received via mailing list)
I faced with buggy behavior of 'QT GUI Label' in my GRC graph and
surprisingly found that GRC generates code making prohibited accesses to
qt
gui objects from non-gui thread context. I'm not python guru, but I
doubt
that python is smart enough to detect access to qt objects from
different
threads and invoke them properly via qt-specific mechanisms. Similar
problem
observed when widgets slots (executing in main thread context) call
variable-changer functions which in turn invoking callbacks of blocks*.

I suppose all blocks in /GUI Widgets/QT category aren't thread-safe and
shouldn't be used until this issue has been fixed.

* Block's external interfaces(callbacks) are expected to be guarded
internally, but I still didn't found no block in gr packages, I working
with, which follow this rule. Maybe swig-generated python interface
guards
all accesses already? If not, GRC user should be so much careful, that
one
have to check generated code. IMHO, it reduces GRC usefullness
greatly...




--
View this message in context:
http://gnuradio.4.n7.nabble.com/Bug-gr-qtgui-wrong...
Sent from the GnuRadio mailing list archive at Nabble.com.
F7b45a110fba31e5b37c60978bc8d3b7?d=identicon&s=25 Artem Pisarenko (Guest)
on 2013-12-26 12:21
(Received via mailing list)
posted partial bugfix to github (pull request #122)



--
View this message in context:
http://gnuradio.4.n7.nabble.com/Bug-gr-qtgui-wrong...
Sent from the GnuRadio mailing list archive at Nabble.com.
C539637020fd56193dd6daec746c4a84?d=identicon&s=25 Tom Rondeau (Guest)
on 2013-12-30 17:28
(Received via mailing list)
On Thu, Dec 26, 2013 at 6:19 AM, Artem Pisarenko <jblackarty@gmail.com>
wrote:
> posted partial bugfix to github (pull request #122)

Artem,

Thanks. I've got this queued up with a few other patches and am just
waiting on confirmation on one more thing (holiday delays) before it
gets pushed.

Tom
Please log in before posting. Registration is free and takes only a minute.
Existing account

NEW: Do you have a Google/GoogleMail, Yahoo or Facebook account? No registration required!
Log in with Google account | Log in with Yahoo account | Log in with Facebook account
No account? Register here.