Closed mherrb closed 4 years ago
Let me verify on my PR2, then I can merge. -Dave
On Thu, Aug 30, 2018 at 8:03 PM Michael Görner notifications@github.com wrote:
@v4hn approved this pull request.
Up to enforce_clock_speed, which we did not discover yet (thank you, less warnings!), we use the same configuration on our PR2.
I hesitated to push this upstream before, because there are quite a number of custom PR2 machines up to now which of course might use different configs anyway. But then there is the "official" indigo servers by Clearpath, so upstream might as well match them.
Thank you @mherrb https://github.com/mherrb , it's great to see upstream PR2 contributions by you these days!
@einig https://github.com/einig for reference.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PR2/pr2_robot/pull/256#pullrequestreview-151246763, or mute the thread https://github.com/notifications/unsubscribe-auth/AFmFudsXCsGKLCHhT5xDT8Ca0Z_9EUfsks5uWKeBgaJpZM4WTgel .
-- David Feil-Seifer, Ph.D. Assistant Professor, Computer Science and Engineering University of Nevada, Reno, 89557 http://cse.unr.edu/~dave office: (775) 784-6469 fax: (775) 784-1877
moved to #263