Text box is not refreshing content on window.
-
@jsulm said in Text box is not refreshing on window.:
@jenya7 said in Text box is not refreshing on window.:
But if m_sensor.DiscoverConcurrent runs - it blocks the text box and I don't see any prints
Sounds like m_sensor.DiscoverConcurrent is a blocking call and does NOT run concurrently.
May be.
uint32_t SENSOR::DiscoverConcurrent(uint8_t min, uint8_t max, uint8_t mode) { QFuture<uint32_t> future; rt_params.service = 1; switch (mode) { case SYS_DISC_MODE_ALL: future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexAll, max); break; case SYS_DISC_MODE_RANGE: future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexRange, min, max); break; case SYS_DISC_MODE_FILE: future = QtConcurrent::run(this, &SENSOR::DiscoverByList); break; } rt_params.service = 0; return future; }At least I try. I get no exceptions or run time messages. How can I be sure another thread was created?
The function is executed - I see result on finish. -
@jsulm said in Text box is not refreshing on window.:
@jenya7 said in Text box is not refreshing on window.:
But if m_sensor.DiscoverConcurrent runs - it blocks the text box and I don't see any prints
Sounds like m_sensor.DiscoverConcurrent is a blocking call and does NOT run concurrently.
May be.
uint32_t SENSOR::DiscoverConcurrent(uint8_t min, uint8_t max, uint8_t mode) { QFuture<uint32_t> future; rt_params.service = 1; switch (mode) { case SYS_DISC_MODE_ALL: future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexAll, max); break; case SYS_DISC_MODE_RANGE: future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexRange, min, max); break; case SYS_DISC_MODE_FILE: future = QtConcurrent::run(this, &SENSOR::DiscoverByList); break; } rt_params.service = 0; return future; }At least I try. I get no exceptions or run time messages. How can I be sure another thread was created?
The function is executed - I see result on finish.@jenya7
I assumeuint32_t SENSOR::DiscoverConcurrent(uint8_t min, uint8_t max, uint8_t mode) { QFuture<uint32_t> future; .... return future; }does an implicit call to result() to return the uint32_t value,
which will wait until the concurrent has run its course.
Congratulations, you turned it back into a synchronous program :P
give the QFuture object to an QFutureWatcher instance and listen to the finished signal there.
Be aware of the Qt Code of Conduct, when posting : https://forum.qt.io/topic/113070/qt-code-of-conduct
Q: What's that?
A: It's blue light.
Q: What does it do?
A: It turns blue. -
@jenya7
I assumeuint32_t SENSOR::DiscoverConcurrent(uint8_t min, uint8_t max, uint8_t mode) { QFuture<uint32_t> future; .... return future; }does an implicit call to result() to return the uint32_t value,
which will wait until the concurrent has run its course.
Congratulations, you turned it back into a synchronous program :P
give the QFuture object to an QFutureWatcher instance and listen to the finished signal there.
@J-Hilk said in Text box is not refreshing content on window.:
does an implicit call to result() to return the uint32_t value
Yes, this is also what I think is happeneing, but can't find anything related to this in QFuture documentation.
-
@J-Hilk said in Text box is not refreshing content on window.:
does an implicit call to result() to return the uint32_t value
Yes, this is also what I think is happeneing, but can't find anything related to this in QFuture documentation.
@jsulm probably this one ?
-
@jsulm said in Text box is not refreshing on window.:
@jenya7 Why is the return type uint32_t and not QFuture<uint32_t>?
I fixed it. But the issue stays.
@jenya7 said in Text box is not refreshing content on window.:
I fixed it
I bet you did not, because discovered variable is still uint32_t, right?
Please read what @J-Hilk wrote. -
@jenya7 said in Text box is not refreshing content on window.:
I fixed it
I bet you did not, because discovered variable is still uint32_t, right?
Please read what @J-Hilk wrote.@jsulm said in Text box is not refreshing content on window.:
@jenya7 said in Text box is not refreshing content on window.:
I fixed it
I bet you did not, because discovered variable is still uint32_t, right?
Please read what @J-Hilk wrote.Now it is
in main.cppQFuture<uint32_t> discovered;in sensor.cpp
QFuture<uint32_t> SENSOR::DiscoverConcurrent()........Still the same.
Wait a minute... SENSOR::DiscoverConcurrent calls
future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexAll, max);so SENSOR::DiscoverByIndexAll - should be also QFuture<uint32_t> ?
well..the problem is only on start up. If I call SENSOR::DiscoverConcurrent() from another place (after return a.exec();) it works good.
-
@jsulm said in Text box is not refreshing content on window.:
@jenya7 said in Text box is not refreshing content on window.:
I fixed it
I bet you did not, because discovered variable is still uint32_t, right?
Please read what @J-Hilk wrote.Now it is
in main.cppQFuture<uint32_t> discovered;in sensor.cpp
QFuture<uint32_t> SENSOR::DiscoverConcurrent()........Still the same.
Wait a minute... SENSOR::DiscoverConcurrent calls
future = QtConcurrent::run(this, &SENSOR::DiscoverByIndexAll, max);so SENSOR::DiscoverByIndexAll - should be also QFuture<uint32_t> ?
well..the problem is only on start up. If I call SENSOR::DiscoverConcurrent() from another place (after return a.exec();) it works good.
-
What if I want to invoke SENSOR::DiscoverConcurrent() after return a.exec();?
What event to connect?QObject::connect(&a, a.???@jenya7 said in Text box is not refreshing content on window.:
What event to connect?
Why do you need an event/signal for that?
Just call it after a.exec()... -
@jenya7 said in Text box is not refreshing content on window.:
What event to connect?
Why do you need an event/signal for that?
Just call it after a.exec()...@jsulm said in Text box is not refreshing content on window.:
@jenya7 said in Text box is not refreshing content on window.:
What event to connect?
Why do you need an event/signal for that?
Just call it after a.exec()...what is "after"?
int main(int argc, char *argv[]) { ............ return a.exec(); discovered = m_sensor.DiscoverConcurrent(0, 4, SYS_DISC_MODE_ALL); }code will never be executed.
-
@jsulm said in Text box is not refreshing content on window.:
@jenya7 said in Text box is not refreshing content on window.:
What event to connect?
Why do you need an event/signal for that?
Just call it after a.exec()...what is "after"?
int main(int argc, char *argv[]) { ............ return a.exec(); discovered = m_sensor.DiscoverConcurrent(0, 4, SYS_DISC_MODE_ALL); }code will never be executed.
@jenya7 said in Text box is not refreshing content on window.:
code will never be executed
Well, it is really not hard to change this code in a way it does what you want.
But I will let this simple exercise to you...
