New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make init.py depend on file conditionally. #1382

Open

zhxchen17 wants to merge 1 commit into NVIDIA:master from zhxchen17:master

zhxchen17 commented Jul 16, 2021

According to https://docs.python.org/3/reference/import.html#__file__ , __file__ is an optional module attribute, and the its availability is decided by the import system. In the edge case where tensorrt is imported as a frozen module (https://docs.python.org/3/c-api/import.html#c.PyImport_ImportFrozenModuleObject), there seems no suitable __file__ attribute to be assigned therefore in this case it's probably better to skip the library loading process.

Signed-off-by: zhxchen17 zhxchen17@fb.com


          Make __init__.py depend on __file__ conditionally.

bb20a6d

According to https://docs.python.org/3/reference/import.html#__file__ , `__file__` is an optional module attribute, and the its availability is decided by the import system. In the edge case where tensorrt is imported as a frozen module (https://docs.python.org/3/c-api/import.html#c.PyImport_ImportFrozenModuleObject), there seems no suitable `__file__` attribute to be assigned therefore in this case it's probably better to skip the library loading process.

Signed-off-by: zhxchen17 <zhxchen17@fb.com>

rajeevsrao assigned pranavm-nvidia

pranavm-nvidia approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment