Closed pierre-rouanet closed 8 years ago
Very Nice !
I would just use 'env.update(m)' instead of 'env.compute_motor_command' plus 'env.compute_sensory_effect' as it wraps both. This appears in the notebook in 'In [5]' and at the end in the goto function.
Also some typos in the notebook:
Maybe put "Reuse of the trained model" instead of "Used the trained models" ?
Rewrite the poppy - explauto bridge (through the use of a PoppyEnvironment) which replaces the previous (and deprecated) ones.
Also add a notebook explaining how this can be used both in simulation and on the real robot.
@sebastien-forestier, @clement-moulin-frier can it be merged?
P.S. BTW, this will be used by students from ENS Rennes pretty soon :-)