search

AllenInstitute / pytic

PyTic - An Object-Oriented Python Wrapper for Pololu Tic Stepper Drivers
Other
10 stars 6 forks source link

yaml.load() without Loader=... is deprecated #5

Open inregards2pluto opened 1 year ago

inregards2pluto commented 1 year ago

Hi! When running tic.settings,load_config(), I kept running into an error message that stated something to the effect of "yml.load(ymlfile) is missing required argument Loader". I determined that the most recent version of PyYAML requires the loader argument and was able to get it working by reverting to version 5.4.1 of PyYAML where the loader argument was optional. Could you either specify the version of PyYAML needed in the readme or update the script so that the loader argument is included in the cfg = yaml.load(ymlfile)? It might be worthwhile to update the script since the following warning popped up when I was running the script:

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

outofculture commented 9 months ago

This is now an error, not just a warning.