search

cbfinn / gps

Guided Policy Search
http://rll.berkeley.edu/gps/
Other
594 stars 239 forks source link

Rostopicsensor #38

Closed cdevin closed 8 years ago

cdevin commented 8 years ago

Changes for the rostopic sensor. The robot plugin was hardcoded to 1 sensor, so to use the rostopic sensor, it must be changed to 2 sensors. I'm not sure whether that should be pushed to everyone.

cdevin commented 8 years ago

Added a commit to not store images in the gui code.

cbfinn commented 8 years ago

Is the memory issue resolved now?

cdevin commented 8 years ago

Yes

On Wed, Jul 6, 2016 at 2:48 PM, Chelsea Finn notifications@github.com wrote:

Is the memory issue resolved now?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/cbfinn/gps/pull/38#issuecomment-230917570, or mute the thread https://github.com/notifications/unsubscribe/ACi-8YavXa62U7AvfeEnXrTmHqgJxQSWks5qTCKkgaJpZM4JFnzU .

cbfinn commented 8 years ago

Ok, merging now then.