Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Undefined name: 'f' --> 'path' #3

Merged
merged 1 commit into from
Dec 2, 2018

Conversation

cclauss
Copy link

@cclauss cclauss commented Dec 2, 2018

f is an undefined name in this context but path is an unused function parameter on the line above.

flake8 testing of https://github.com/facebookresearch/VideoPose3D on Python 3.7.1

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./data/data_utils.py:91:20: F821 undefined name 'f'
    with h5py.File(f) as hf:
                   ^
1     F821 undefined name 'f'
1

__f__ is an _undefined name_ in this context but __path__ is an unused function parameter on the line above.

[flake8](http://flake8.pycqa.org) testing of https://github.com/facebookresearch/VideoPose3D on Python 3.7.1

$ __flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics__
```
./data/data_utils.py:91:20: F821 undefined name 'f'
    with h5py.File(f) as hf:
                   ^
1     F821 undefined name 'f'
1
```
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Dec 2, 2018
@michaelauli michaelauli merged commit cf94b42 into facebookresearch:master Dec 2, 2018
@michaelauli
Copy link
Contributor

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants