Update osim.py

[gautam1858]

Here at each time you are creating different random targets, so changing it to make a single target so that the arm can try to reach the specified target position

[kidzik]

Thank you for your contribution @gautam1858. The point of the generate_new_target function is really to set a new random target. If you don't want to change targets after reset then it's probably better to parametrize the reset function https://github.com/stanfordnmbl/osim-rl/blob/master/osim/env/osim.py#L535-L539

to something like

def reset(self, random_target = True):
    obs = super(Arm2DEnv, self).reset()
    if random_target:
        self.generate_new_target()
    self.osim_model.reset_manager()
    return obs

and then in your code yau can call env.reset(random_target = False)

[gautam1858]

ok cool :)