You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/07/08 17:10:57 UTC

[GitHub] [beam] yeandy commented on a diff in pull request #22131: TensorRT Initial commit

yeandy commented on code in PR #22131:
URL: https://github.com/apache/beam/pull/22131#discussion_r917002315


##########
sdks/python/apache_beam/ml/inference/tensorrt_inference.py:
##########
@@ -0,0 +1,213 @@
+#
+# SPDX-FileCopyrightText: Copyright (c) 2022 NVIDIA CORPORATION & AFFILIATES. All rights reserved. # pylint: disable=line-too-long

Review Comment:
   @azhurkevich Does `SPDX-FileCopyrightText: Copyright (c) 2022 NVIDIA CORPORATION & AFFILIATES. All rights reserved.` have to be on one line? I made some formatting fixes, and added `# pylint: disable=line-too-long` for now to keep it all on one line.
   
   But if it can be broken up into two lines, then it'll look better if we make that change.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org