Resolve "Add a battery widget"

Closes #54 (closed)

assigned to @hahiho

added 11 commits

• b22706ed...c7e6fdd6 - 11 commits from branch master

Compare with previous version

added 1 commit

• e2487fa3 - Create BatteryWidget class

Compare with previous version

added 1 commit

• 3a530a5e - Implement first draft for battery icon widget

Compare with previous version

added 1 commit

• 84c015ea - Make Battery Widget visible

Compare with previous version

added 1 commit

• d91a1d59 - Cleanup

Compare with previous version

added 1 commit

• bead9e1e - Small hack to keep symmetry in same cases (e.g. in case of toolbar)

Compare with previous version

added 1 commit

• 81283dba - Implement icon for unavailable battery und display actual data from armar 3 simulation

Compare with previous version

added 1 commit

• e01d3a80 - Use correct datafields for battery widget (not working, but no idea why)

Compare with previous version

By ARMAR-6 on 2022-06-27T14:09:37 (imported from GitLab)

added 1 commit

• 08176e74 - Use fixed size for battery widget

Compare with previous version

added 1 commit

• a6be50b8 - Fix some bugs in battery widget and improve appearance

Compare with previous version

By ARMAR-6 on 2022-06-29T12:46:56 (imported from GitLab)

added 1 commit

• 36ef4fa3 - More efficient implementation to get current battery state

Compare with previous version

By ARMAR-6 on 2022-06-29T13:30:16 (imported from GitLab)

added 1 commit

• f4b00389 - ...

Compare with previous version

By ARMAR-6 on 2022-06-29T13:43:25 (imported from GitLab)

added 1 commit

• 47da5d1e - BatteryWidget: Implement tooltip

Compare with previous version

By ARMAR-6 on 2022-07-04T10:30:45 (imported from GitLab)

added 1 commit

• 59aa937c - Battery Widget: More precise drawing

Compare with previous version

By ARMAR-6 on 2022-07-04T10:36:48 (imported from GitLab)

added 1 commit

• 4bd15be6 - Less hacky drawing code in battery widget

Compare with previous version

added 1 commit

• de103d08 - BatteryWidget: Re-introduce changes previously made in drawing code

Compare with previous version

added 1 commit

• ca54f43c - BatteryWidget: Trigger updates in a background thread

Compare with previous version

added 1 commit

• 7e55ef33 - BatteryWidget: Fix double free bug

Compare with previous version

added 1 commit

• bf275516 - BatteryWidget: Make methods public if they are called from QMetaObject::invokeMethod

Compare with previous version

added 1 commit

• b9efef89 - BatteryWidget: Some minor visual improvements

Compare with previous version

added 1 commit

• 7f99e0ef - BatteryWidget: Remove unnecessary includes

Compare with previous version

added 1 commit

• 5288da0f - BatteryWidget: Correct package name

Compare with previous version

added 7 commits

• 5288da0f...c15250a1 - 6 commits from branch master
• f326fd9c - Merge branch 'master' into battery_widget

Compare with previous version

added 1 commit

• f6559e3d - BatteryWidget: Remove more unnecessary stuff

Compare with previous version

added 1 commit

• 569fe18c - BatteryWidget: Remove dead code comments

Compare with previous version

marked this merge request as ready

requested review from @dreher

Altogether it works good, however there are a lot of error messages generated in the case of robot reconnects etc. This should be handled, since it confuses people.

There are two types of messages. The error messages which are printed automatically whenn I read sth from the observer that does not exist. They need to be fixed elsewhere. And the debug message thet my code prints every 20s or so if it has no connection.

Or maybe there is another way to access observers without generating error messages?

added 15 commits

• 569fe18c...0f379fa5 - 14 commits from branch master
• d2f518ba - Merge remote-tracking branch 'origin/master' into battery_widget

Compare with previous version

As discussed with @kartmann: We mute the errors and warnings from the observer once and for all. The errors must be propagated so code can actually do something about it.

If you know where to look, you can do it Niklas and then we can finally merge this!

resolved all threads

added 32 commits

• d2f518ba...215be30b - 31 commits from branch master
• 168bd53d - Merge remote-tracking branch 'origin/master' into battery_widget

Compare with previous version

Has this been merged now? Because I saw the GUI on ARMAR-7? @reister?

mentioned in merge request !102 (merged)

merged manually

Merged via !102 (merged) using a dedicated interface instead of observers

Thanks for your work @ujrwq_arlt, it really looks nice! Will be rolled out on ARMAR-7 as soon as it is available again.