Skip to content

Conversation

@chriselion
Copy link
Contributor

This gets rid of these warnings

/.../ml-agents/ml-agents/mlagents/trainers/learn.py:157: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
trainer_config = yaml.load(data_file)

when running mlagents-learn. It's also just a generally good idea; see the link for more info.

@chriselion
Copy link
Contributor Author

Tests failing until #2125 is merged in

@chriselion chriselion requested a review from xiaomaogy June 12, 2019 00:55
@chriselion chriselion mentioned this pull request Jun 12, 2019
@chriselion chriselion merged commit 9c354a6 into develop Jun 12, 2019
@chriselion chriselion mentioned this pull request Jun 12, 2019
@chriselion chriselion deleted the develop-yaml-safe_load branch July 11, 2019 22:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants