amandlek
amandlek
Only saving successful demonstrations should already be the default (see [this line](https://github.com/ARISE-Initiative/robosuite/blob/master/robosuite/scripts/collect_human_demonstrations.py#L72)). It would be helpful if you could provide some more details on how we can reproduce this issue...
This is a great catch! You're right - by default all demonstrations are saved, including the ones that are unsuccessful. The easiest way to fix this would be to modify...
Great catch - thanks for reporting this bug! We will PR a fix soon.
Would you be able to send over a trained model on one of our datasets so that we can reproduce this issue easily? Or even confirm that it breaks with...
Yup, you're right - we used the same trick during demo collection to ensure that the demonstrations could be deterministically played back on the same machine that was used for...
I think it removes some numerical inaccuracies that might come from reading the state from the simulator and then setting it, but I'm not 100% on this. It's incredibly strange...
Closing this issue as it appears to be stale, and also more related to [robosuite](https://github.com/ARISE-Initiative/robosuite). Feel free to raise an issue there if problems persist.
We need more information to reproduce the issue. Can you give us the precise training command, config being used, and information on the dataset?
Not at the moment.