vwxyzjn / cleanrl

High-quality single file implementation of Deep Reinforcement Learning algorithms with research-friendly features (PPO, DQN, C51, DDPG, TD3, SAC, PPG)

Home Page:http://docs.cleanrl.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Video upload Issue - wandb

tbasaklar opened this issue · comments

Hi,

I was following the "Experiment Tracking" at https://docs.cleanrl.dev/get-started/experiment-tracking/

In this example, the captured videos are also uploaded to wandb. However, although I followed the exact same command, my wandb project missed the "Media" folder in "Files", hence the videos do not show up in the browser. What could be the issue here? Could you try to rerun the command in the docs page of cleanRL and see if the videos are uploaded on your side?

Thanks,

You can probably do this

if args.track and args.capture_video:

there are some ongoing gymnasium migration stuff that might have broken the Wandb gym integration.

I've also realized that if I change the command from:

python cleanrl/ppo.py --seed 1 --env-id CartPole-v1 --track --capture-video

to

python cleanrl/dqn.py --seed 1 --env-id CartPole-v1 --track --capture-video

It works! I wondered why that happened and realized it was because of the gymnasium migration, as you mentioned.
For dqn.py, the migration to the gymnasium is completed, I guess, and that's why it works.

I've also added the following condition (in ppo_continuous_action.py) to ppo.py. It works now!

Thanks,

Hi,

I appreciate the work, and I really love to work with cleanRL, especially for research purposes. When do you think you can complete the gymnasium migration for all algorithms? It is hard to follow the examples when they give errors related to Gymnasium and Gym discrepancies.

Best,

#424 should have completed the migration for almost all.