pyyaml warning for missing Loader argument

DataBiosphere / dsub

Open-source command-line tool to run batch computing tasks and workflows on backend services such as Google Cloud.

Apache License 2.0

265 stars 44 forks source link

pyyaml warning for missing Loader argument #150

Closed BenjaminDoran closed 5 years ago

BenjaminDoran commented 5 years ago

Whenever I run dsub with yaml version '5.1'. I get this warning:

/anaconda3/envs/dsub/lib/python2.7/site-packages/dsub/lib/job_model.py:797: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details

It would be nice if this warning was fixed for later releases of dsub. Seems like calls to yaml.load() should now specify a loader.

mbookman commented 5 years ago

Thanks for reporting this Benjamin. A fix for this is in progress.

mbookman commented 5 years ago

This is fixed in release 0.3.1 by e84fc39b3fd7fd969072483d0935532a6c292996.