Incorrect reset space for object in disassemble

[ottofabian]

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.

[reginald-mclean]

Hi @ottofabian I cannot replicate this issue (it's possible PR #370 fixed this). As Meta-World ownership has been transferred from the RLWorkGroup to Farama Foundation, I am closing this issue. If there are any questions/issues please join our Discord

[ottofabian]

Hi @reginald-mclean. The referenced PR might have fixed this bug as it depends how you generate the uniform samples.
Regardless, I would still propose to change the upper and lower bound of the first elements here to avoid issues like this in the future

[reginald-mclean]

Oh sorry, I misread the original issue. Thanks for pointing out that issue!