search

openvax / mhcflurry

Peptide-MHC I binding affinity prediction
http://openvax.github.io/mhcflurry/
Apache License 2.0
191 stars 57 forks source link

Add Loader to yaml.load call #141

Closed susannasiebert closed 5 years ago

susannasiebert commented 5 years ago

This fixes the calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe warning messages.

See https://msg.pyyaml.org/load.

coveralls commented 5 years ago

Coverage Status

Coverage decreased (-0.05%) to 43.889% when pulling 793d8cb5824c94f1ce4de3772f330aff9fb3d383 on susannasiebert:yaml_load into 35fd95fab4350e1b50e081414ede6ae27ca15395 on openvax:master.

timodonnell commented 5 years ago

Appreciate this, @susannasiebert ! I made a small variant of this PR in #142 which fixes the warning issue by calling yaml.safe_load and also fixes these travis failures. Closing this PR in favor of #142.

Many thanks for your contributions!

timodonnell commented 5 years ago

Pushed version 1.2.4 with these changes to PyPI: https://pypi.org/project/mhcflurry/

susannasiebert commented 5 years ago

You're welcome. Thanks for the expedient new release.