-
Notifications
You must be signed in to change notification settings - Fork 183
DQN mnist & mountain car performance #20
Comments
Hi Peter! Thanks for raising this... I think we might have seen some slippage in agent performance. I'm not sure if this has come from updates to:
My suspicion is there are some small details in this migration TF1->TF2 that changed some scores (the agents aren't exactly the same). Many thanks, |
Hello again! I have just run the agents checked in at HEAD and I did not see your observed scoring... We may need to tool in some more continuous testing, but the scores on mnist in particular seem "off" for the DQN implementation. Can you confirm this is still an issue for you? |
I also have a similar observation concerning MountainCar, however it is related to the AC algorithm. To me it seems like there is a major difference between the results reported in the paper (close to 1) versus the ones in this thread (close to 0). I have also tried running actor_critic_rnn on mountain_car and it does not seem to learn (on default hyper params). |
Yes @mklissa - I see that difference above. There have been several moving pieces:
However, I think the best approach is to go from what is at HEAD and start a new issue to update paper/reference colabs to incorporate this bug fix. |
Dear Ian, Thanks for looking into this! Back in march, I observed poor performance on mnist with both the baseline implementation as well as my own implementation of DQN. Given that mnist seems to work perfectly fine for you, I assume there must be some problem on my side. I will set up a system from scratch and run the baseline implementation of dqn again. It might take a while though until I find time to do that. Best regards, Peter |
Hi, I used a fresh install of pop!_os 20.04 (distribution based on Ubuntu) and then performed as few steps as possible to run the agent:
Hope this helps. Best regards, |
Ah... OK well I think in order to get the claimed performance, you need to run the dqn.default_agent() I can see that this is a bit confusing, but we wanted to expose the flags as an easy way for people to tinker! BTW... do you think we should instead remove the flag options and avoid this kind of confusion? |
I would keep the flag options but maybe have the same values as the default agent as default. As you suggested, I replaced the agent (that uses the flags) with the dqn.default_agent() in run.py and ran the experiments again. Unfortunately no improvement on the mnist experiments. @mklissa you said you observed something similar on MountainCar. Did the mnist experiments work for you? If I'm the only one experiencing this problem, then there might just be some issue on my side. Best regards, |
Hey there, I'm writing to report that I'm also experiencing the same problem as @pluebcke in MNIST. I couldn't replicate the good MNIST results as reported in the paper. I also noticed a bad performance (0.2-0.26 score at most) using PPO and DQN agents from an external library (stable-baselines), tried different hyper-parameters, number of layers/neurons, activation functions, with no effect. I also checked the MNIST env implementation offered here and seemed OK to me. Today I created a new virtual env with the latest BSuite version with the baselines, runned the 20 seeds twice and the baseline DQN agent also scored 0.23. This also happens with noise and scale variants. |
Hi @jbarsce - I'm not sure I understand the question. So, are you saying that: We have some tools for testing this internally within Google/DeepMind... and based on that I'm confident that the bsuite/baselines/jax/dqn and bsuite/baselines/tf/dqn do reproduce the performance. However... we clearly need to work out a way to share these tests/reproducibility/installation instructions so that this confusion does not arise. |
Hi Ian, thanks for the quick reply! yes, I ran the BSuite experiments with another DQN agent and noticed that, while the other envs performed similar than in the accompanying paper, MNIST was the only that underperformed. As this external agent had several variations, I tried to replicate the results with the DQN agent from this repo, trying tf and jax and isolating them in a new virtual environment. In case they are of any help, the following were the steps I followed (I took them from the jax repo and from here)
For tensorflow 2.1 I ran the experiments with
For jax
Environment: Ubuntu 18.04 bionic Please let me know if you need any other information. Finally, thanks for this great repository |
Just a wild guess, maybe something went wrong with the download of the input mnist dataset for Juan and me? |
Yes interesting... something is getting lost between the version that is checked in to Google3 and the settings you are running. @yotam @aslanides and I will have a look into this... Going to keep this open for now and try to reproduce this... |
Repro in ~10 lines (excluding imports): https://colab.research.google.com/drive/1XtTv-p2bXfvMBT_77cWjWRHPXIvimWlO?usp=sharing |
Hi, |
Hi,
while working on a PyTorch DQN agent for BSuite experiments, I noticed quite bad results on the mnist and mountain car experiments. I see that a similar question was addressed here, but the thread was closed.
To further investigate, I created a new conda environment, downloaded and installed a fresh copy of BSuite and ran the DQN agent from the baselines. The only settings I've changed were "bsuite_id" to "SWEEP" and the save path.
When you compare the results from both agents with the barplot on page 16 of the BSuite manuscript, you notice that both agents have worse performance on mnist and mountaincar and better performance on catch.
Were there any changes on the environments that I missed? The DQN agent from the manuscript did use the default parameters from the baseline directory, correct?
Thanks,
Peter
The text was updated successfully, but these errors were encountered: