search

ajwdewit / pcse

Repository for the Python Crop Simulation Environment
Other
184 stars 125 forks source link

yaml safe load #50

Closed S-lac closed 2 years ago

S-lac commented 2 years ago

Hi and thanks for the great package! I have been having a warning for a long time now, even though it doesn't result in any problem in simulations :

For : \pcse\fileinput\yaml_agro_loader.py:21

YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.

Replacing yaml.load() to yaml.safe_load() might to the trick if I am not mistaken.

ajwdewit commented 2 years ago

You are correct. It is fixed in the develop branch. I will upgrade to PCSE 5.5 in the coming weeks which will solve it.