search

sbgisen / moveit

:robot: The MoveIt motion planning framework
http://moveit.ros.org/
BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

CPU usage of rviz increases when running for long periods #8

Closed nyxrobotics closed 2 months ago

nyxrobotics commented 1 year ago

Summary Since the 814908b pull request, the cpu usage of rviz has been increasing over time. I went back and verified the commits one by one, it seems to be caused by commit d98a529

Reproduction Procedure Start rviz, moveit, and the gazebo environment.

How to fix it Revert d98a529

Cause I don't know what this commit is needed for or why this caused the bug It's hard to see the commit log

Proposed fix Revert d98a529

nyxrobotics commented 1 year ago

I'd like to hear the official opinion, so I've set up an issue in the original repository https://github.com/ros-planning/moveit/issues/3472#issue-1785198475

jsupratman13 commented 10 months ago

https://github.com/ros-planning/moveit/pull/3478

jsupratman13 commented 10 months ago

since the main repository already have a patch to this problem, we should upstream our repository too

nyxrobotics commented 10 months ago

9 imported upstream

jsupratman13 commented 10 months ago

my bad didn't notice it