{"action":"created","issue":{"url":"https://api.github.com/repos/ros-planning/navigation/issues/710","repository_url":"https://api.github.com/repos/ros-planning/navigation","labels_url":"https://api.github.com/repos/ros-planning/navigation/issues/710/labels{/name}","comments_url":"https://api.github.com/repos/ros-planning/navigation/issues/710/comments","events_url":"https://api.github.com/repos/ros-planning/navigation/issues/710/events","html_url":"https://github.com/ros-planning/navigation/issues/710","id":316511227,"number":710,"title":"Thread safety in costmap_2d costmap reset service","user":{"login":"SteveMacenski","id":14944147,"avatar_url":"https://avatars3.githubusercontent.com/u/14944147?v=4","gravatar_id":"","url":"https://api.github.com/users/SteveMacenski","html_url":"https://github.com/SteveMacenski","followers_url":"https://api.github.com/users/SteveMacenski/followers","following_url":"https://api.github.com/users/SteveMacenski/following{/other_user}","gists_url":"https://api.github.com/users/SteveMacenski/gists{/gist_id}","starred_url":"https://api.github.com/users/SteveMacenski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SteveMacenski/subscriptions","organizations_url":"https://api.github.com/users/SteveMacenski/orgs","repos_url":"https://api.github.com/users/SteveMacenski/repos","events_url":"https://api.github.com/users/SteveMacenski/events{/privacy}","received_events_url":"https://api.github.com/users/SteveMacenski/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-04-21T17:51:28Z","updated_at":"2018-04-23T18:38:34Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"My colleague and I found the clear costmaps service does not lock the costmap mutex as `updateBounds` and the `clear_costmap_recovery` do. Would adding this lock be something that would be merged?\r\n\r\nWe find the robot is able to plan in some states when this is called frequently, it will generate a straight line path through the map because there's no costs represented in that iteration."},"comment":{"url":"https://api.github.com/repos/ros-planning/navigation/issues/comments/383678888","html_url":"https://github.com/ros-planning/navigation/issues/710#issuecomment-383678888","issue_url":"https://api.github.com/repos/ros-planning/navigation/issues/710","id":383678888,"user":{"login":"SteveMacenski","id":14944147,"avatar_url":"https://avatars3.githubusercontent.com/u/14944147?v=4","gravatar_id":"","url":"https://api.github.com/users/SteveMacenski","html_url":"https://github.com/SteveMacenski","followers_url":"https://api.github.com/users/SteveMacenski/followers","following_url":"https://api.github.com/users/SteveMacenski/following{/other_user}","gists_url":"https://api.github.com/users/SteveMacenski/gists{/gist_id}","starred_url":"https://api.github.com/users/SteveMacenski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SteveMacenski/subscriptions","organizations_url":"https://api.github.com/users/SteveMacenski/orgs","repos_url":"https://api.github.com/users/SteveMacenski/repos","events_url":"https://api.github.com/users/SteveMacenski/events{/privacy}","received_events_url":"https://api.github.com/users/SteveMacenski/received_events","type":"User","site_admin":false},"created_at":"2018-04-23T18:38:34Z","updated_at":"2018-04-23T18:38:34Z","author_association":"CONTRIBUTOR","body":"The changes are in PR #711, its just a couple locks in the service call so I'd doubt anyone would adamantly object. \r\n\r\nI'll check back up on this in a week or two to check in. "}}
{"actor":{"display_login":"SteveMacenski"}}