Skip to content

Delete redundant source code#49

Merged
Tabrizian merged 1 commit into
triton-inference-server:mainfrom
qpakzk:redundancy
May 11, 2021
Merged

Delete redundant source code#49
Tabrizian merged 1 commit into
triton-inference-server:mainfrom
qpakzk:redundancy

Conversation

@qpakzk

@qpakzk qpakzk commented May 6, 2021

Copy link
Copy Markdown
Contributor

Two varaibles api_version_major and api_version_minor are already initialized in line 1029.

* api_version_major and api_version_minor are already initialized in line 1029

Signed-off-by: Sangwon Hong <qpakzk@gmail.com>

@CoderHam CoderHam left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@qpakzk thanks for the contribution. Is this your first contribution? You might need to sign a CLA in that case.
@Tabrizian this changeslook good to me, I am approving it. Feel free to chime in an trigger a CI run for the same

@Tabrizian Tabrizian self-requested a review May 6, 2021 13:17
@Tabrizian

Copy link
Copy Markdown
Member

Please sign the CLA and after that we can merge this.

https://github.com/triton-inference-server/server/blob/master/CONTRIBUTING.md

@qpakzk

qpakzk commented May 7, 2021

Copy link
Copy Markdown
Contributor Author

@CoderHam @Tabrizian Hello.
I sent the CLA to triton-cla@nvidia.com.

@Tabrizian Tabrizian merged commit ca74b59 into triton-inference-server:main May 11, 2021
@qpakzk qpakzk deleted the redundancy branch May 13, 2021 05:24
baojunliu pushed a commit to baojunliu/python_backend that referenced this pull request Apr 22, 2022
- Some models outputs can be on a device different than the model itself
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants