Skip to content

Resolved YAMLLoadWarning adding Loader=FullLoader#11

Closed
Elevate08 wants to merge 1 commit into
chestm007:masterfrom
Elevate08:master
Closed

Resolved YAMLLoadWarning adding Loader=FullLoader#11
Elevate08 wants to merge 1 commit into
chestm007:masterfrom
Elevate08:master

Conversation

@Elevate08
Copy link
Copy Markdown

When running amdgpu-fan from command line there is a YAMLLoadWarning when using python3.7. This was resolved by adding Loader=FullLoader on line 44

@tawera
Copy link
Copy Markdown

tawera commented Mar 10, 2020

I suspect an import statement is missing:

Traceback (most recent call last): File "/usr/local/bin/amdgpu-fan", line 11, in <module> load_entry_point('amdgpu-fan===PROJECTVERSION', 'console_scripts', 'amdgpu-fan')() File "/usr/local/lib/python3.7/site-packages/amdgpu_fan-PROJECTVERSION-py3.7.egg/amdgpu_fan/controller.py", line 67, in main File "/usr/local/lib/python3.7/site-packages/amdgpu_fan-PROJECTVERSION-py3.7.egg/amdgpu_fan/controller.py", line 44, in load_config NameError: name 'FullLoader' is not defined
Either add import or use full_load or safe_load instead of load:

https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation#how-to-disable-the-warning

@tawera
Copy link
Copy Markdown

tawera commented Mar 12, 2020

I see now that #13 is a duplicate of this PR which solves the same issue the correct way so please merge that one and close this one.

@Elevate08 Elevate08 closed this Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants