[Minor Bug] Unnecessary and mildy confusing print statement during hyperparameter optimization

Starting a hyperparameter run with train.py prints out something like this:

========== LunarLanderContinuous-v2 ==========
Seed: 3139539977
OrderedDict([('batch_size', 64),
             ('ent_coef', 0.01),
             ('gae_lambda', 0.98),
             ('gamma', 0.999),
             ('n_envs', 16),
             ('n_epochs', 4),
             ('n_steps', 1024),
             ('n_timesteps', 1000000.0),
             ('policy', 'MlpPolicy')])
Using 4 environments
Overwriting n_timesteps with n=4000000

The problem is that the OrderedDict that’s printed is the parameters for the environment in the relevant .yml file. It has no impact on hyperparameter tuning, isn’t even used, can make it look like you’re doing something wrong (e.g. by default you’re tuning hyperparameters that aren’t included in that list) and the origin of it takes a shockingly long time to hunt down to confirm you aren’t somehow doing something wrong.

[jkterry1]

(sorry for the delayed reply)

I meant a message like “Default hyperparameters for environment (ones being tuned will be overridden)”"

[araffin]

I meant a message like “Default hyperparameters for environment (ones being tuned will be overridden)”"

sounds good 😉 (my upvote did not create a notification …)