diff --git a/ml-agents/mlagents/trainers/learn.py b/ml-agents/mlagents/trainers/learn.py index 13e7edd3b7..6a4f23e13d 100644 --- a/ml-agents/mlagents/trainers/learn.py +++ b/ml-agents/mlagents/trainers/learn.py @@ -154,7 +154,7 @@ def prepare_for_docker_run(docker_target_name, env_path): def load_config(trainer_config_path): try: with open(trainer_config_path) as data_file: - trainer_config = yaml.load(data_file) + trainer_config = yaml.safe_load(data_file) return trainer_config except IOError: raise UnityEnvironmentException( diff --git a/ml-agents/mlagents/trainers/tests/test_bc.py b/ml-agents/mlagents/trainers/tests/test_bc.py index d3166fe22d..800ab4a23f 100644 --- a/ml-agents/mlagents/trainers/tests/test_bc.py +++ b/ml-agents/mlagents/trainers/tests/test_bc.py @@ -13,7 +13,7 @@ @pytest.fixture def dummy_config(): - return yaml.load( + return yaml.safe_load( """ hidden_units: 128 learning_rate: 3.0e-4 diff --git a/ml-agents/mlagents/trainers/tests/test_ppo.py b/ml-agents/mlagents/trainers/tests/test_ppo.py index 716bd8702b..67c9948d2e 100644 --- a/ml-agents/mlagents/trainers/tests/test_ppo.py +++ b/ml-agents/mlagents/trainers/tests/test_ppo.py @@ -14,7 +14,7 @@ @pytest.fixture def dummy_config(): - return yaml.load( + return yaml.safe_load( """ trainer: ppo batch_size: 32 diff --git a/ml-agents/mlagents/trainers/tests/test_trainer_controller.py b/ml-agents/mlagents/trainers/tests/test_trainer_controller.py index 5e5a5e75a5..ed97024a3f 100644 --- a/ml-agents/mlagents/trainers/tests/test_trainer_controller.py +++ b/ml-agents/mlagents/trainers/tests/test_trainer_controller.py @@ -16,7 +16,7 @@ @pytest.fixture def dummy_config(): - return yaml.load( + return yaml.safe_load( """ default: trainer: ppo @@ -46,7 +46,7 @@ def dummy_config(): @pytest.fixture def dummy_online_bc_config(): - return yaml.load( + return yaml.safe_load( """ default: trainer: online_bc @@ -78,7 +78,7 @@ def dummy_online_bc_config(): @pytest.fixture def dummy_offline_bc_config(): - return yaml.load( + return yaml.safe_load( """ default: trainer: offline_bc @@ -120,7 +120,7 @@ def dummy_offline_bc_config_with_override(): @pytest.fixture def dummy_bad_config(): - return yaml.load( + return yaml.safe_load( """ default: trainer: incorrect_trainer