Skip to content

Conversation

@narendasan
Copy link
Collaborator

Signed-off-by: Naren Dasan [email protected]
Signed-off-by: Naren Dasan [email protected]

@narendasan narendasan merged commit 808e603 into master Oct 20, 2021
@narendasan narendasan deleted the python_packaging_feats branch October 20, 2021 17:59
@github-actions github-actions bot added the component: api [Python] Issues re: Python API label Oct 20, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some changes that do not conform to Python style guidelines:

--- /workspace/py/setup.py	(original)
+++ /workspace/py/setup.py	(reformatted)
@@ -24,14 +24,15 @@

__version__ = '0.5.0a0'

+
def get_git_revision_short_hash() -> str:
    return subprocess.check_output(['git', 'rev-parse', '--short', 'HEAD']).decode('ascii').strip()
+

if "--release" not in sys.argv:
    __version__ = __version__ + "+" + get_git_revision_short_hash()
else:
    sys.argv.remove("--release")
-

if "--use-cxx11-abi" in sys.argv:
    sys.argv.remove("--use-cxx11-abi")
Reformatting /workspace/py/setup.py
Reformatting /workspace/py/trtorch/logging.py
Reformatting /workspace/py/trtorch/_compile_spec.py
Reformatting /workspace/py/trtorch/__init__.py
Reformatting /workspace/py/trtorch/ptq.py
Reformatting /workspace/py/trtorch/Input.py
Reformatting /workspace/py/trtorch/_types.py
Reformatting /workspace/py/trtorch/_compiler.py
Reformatting /workspace/py/trtorch/Device.py
ERROR: Some files do not conform to style guidelines

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code conforms to C++ style guidelines

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

Labels

component: api [Python] Issues re: Python API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants