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

rqt_bag:thumbnail toggle causes crash #380

Closed
hugbed opened this issue Jun 13, 2016 · 2 comments
Closed

rqt_bag:thumbnail toggle causes crash #380

hugbed opened this issue Jun 13, 2016 · 2 comments
Labels

Comments

@hugbed
Copy link
Contributor

hugbed commented Jun 13, 2016

Traceback (most recent call last):
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rqt_bag/timeline_frame.py", line 280, in paint
    self._draw_topic_histories(painter)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rqt_bag/timeline_frame.py", line 384, in _draw_topic_histories
    self._draw_topic_history(painter, topic)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rqt_bag/timeline_frame.py", line 464, in _draw_topic_history
    renderer.draw_timeline_segment(painter, topic, stamp_start, stamp_end, region_x_start, msg_y, region_width, msg_height)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rqt_bag_plugins/image_timeline_renderer.py", line 130, in draw_timeline_segment
    painter.setPen(QPen(QBrush(Qt.black)))
TypeError: QPen(): argument 1 has unexpected type 'QBrush'

Toggling the thumbnails in rqt_bag causes a crash in image_timeline_renderer.py.
Seems like the constructor of Qt.black should have Qt.black and not QBrush(Qt.black).

dirk-thomas pushed a commit that referenced this issue Jun 27, 2016
Fixes the crash in image_timeline_renderer.py when toggling thumbnails in rqt_bag.
Seems like the constructor for QPen expects a QColor and not a QBrush.
@dirk-thomas
Copy link
Contributor

Addressed by #381.

@ubuntuslave
Copy link

ubuntuslave commented Aug 11, 2016

I was getting the same issue on Ubuntu 16.4 and ROS kinectic. Since the debian package for ros-kinectic-rqt-common-pluggins is still outdated, I had to overlay a workspace to install from source.

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

No branches or pull requests

3 participants