Craig Sherstan

Results 13 issues of Craig Sherstan

To ensure good performance, all serial connections should be set to low latency. We should add a check for this for Dynamixels and Creates. To check latency of a particular...

The DXL and UR5 environments have been updated to move the communicator setups outside the environments, instead they are now passed in as a parameter. The same needs to be...

While doing some testing it was observed that I could get better performance at short cycle times (< 20 ms) using a custom written communicator than from using SenseAct's DXLCommunicator....

The DXL and Create environments have a lot of duplicated code. It would be great to combine these.

Both DxlReacher1DEnv and DxlTracker1DEnv use a custom PID controller to return the motor to the starting position. This is unnecessary - it is straightforward to switch to position control. However,...

This file offers some useful helper functions that are higher level than the driver, but not wrapped in SenseAct's Communicator. However, they are tied to a specific port, baud, etc....

There was a refactor in baselines which changed the function signatures: https://github.com/openai/baselines/commit/8c2aea2addc9f3ba36d4a0c937e6a2d09830afc7 I've only looked at the dxl code, but it still uses the previous arguments and won't run with...

I'm just going to dump a number of points I had written down about the wiki. I had to install a number of additional packages: - python3-pip - setuptools -...

dxl_basic_functions.py is written in such a way the you have to go in and tweak the code by hand to set the port, motor id, etc. I suggest it should...

MX64AT. Try this: ```bulk_read_test([1])``` Result: > Traceback (most recent call last): File "/usr/lib/python3.5/code.py", line 91, in runcode exec(code, self.locals) File "", line 1, in File "/home/craig/workspace/SenseAct/senseact/devices/dxl/dxl_basic_functions.py", line 239, in bulk_read_test...