Metaworld icon indicating copy to clipboard operation
Metaworld copied to clipboard

Incorrect reset space for object in disassemble

Open ottofabian opened this issue 1 year ago • 0 comments

The lower bound for the random reset space of the object in disassemble is higher than the upper bound for the first index. This appears to be an issue for both v1 https://github.com/rlworkgroup/metaworld/blob/18118a28c06893da0f363786696cc792457b062b/metaworld/envs/mujoco/sawyer_xyz/v1/sawyer_disassemble_peg.py#L14-L15 and v2 https://github.com/rlworkgroup/metaworld/blob/18118a28c06893da0f363786696cc792457b062b/metaworld/envs/mujoco/sawyer_xyz/v2/sawyer_disassemble_peg_v2.py#L15-L16

np.random.uniform seems to invert the values in case high > low. However when using explicit seeding with self.np_random, seed = seeding.np_random(seed) self.np_random.uniform raises a ValueError.

ottofabian avatar Jul 07 '22 07:07 ottofabian