-
Notifications
You must be signed in to change notification settings - Fork 583
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Lack of image normalization in DiffusionPolicy #48
Comments
@hesic73 same question,and whether the normalization is added or not, the final eval performance of Diffusion policy is poor,arms keeps shaking and can't complete the task. Have you solved this problem? |
@woltium Hello, How did Diffusion Policy even run for you? Even after following all the instructions I get an error until I turn off ENABLE_EMA = True in policy.py to false. Once I do this though, I receive " raise _control.PhysicsError(message) |
@woltium I used this command: python3 imitate_episodes.py --task_name sim_transfer_cube_scripted --ckpt_dir /home/Desktop/learning --policy_class Diffusion --chunk_size 32 --batch_size 32 --lr 1e-5 --seed 0 --num_steps 100000 --eval_every 2000 --validate_every 2000 --save_every 2000 |
In
policy.py
, image normalization is applied in both ACTPolicy and CNNMLPPolicy as follows:However, this normalization step is missing in DiffusionPolicy. Is this omission intentional? Could you clarify the reasoning behind this difference?
The text was updated successfully, but these errors were encountered: