Skip to content
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.

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

Python InferInput: set_data_from_numpy is not necessary in some cases. #194

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

RRoundTable
Copy link

Objective

In python, InferInput only support set_data_from_numpy. It means that users have to convert their data to numpy array even though it it not necessary.

So, I added an interface that can set raw_content without converting to numpy array.

Detail

When model in triton server can process data that has bytes datatype, User don't have to converting it to numpy array.

For example, if we use dali as preprocessor, dali can process bytes data.

@matthewkotila
Copy link
Contributor

Hi @RRoundTable, thanks for making these changes. Can you fill out and email the CLA as described here?:

https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

@RRoundTable
Copy link
Author

Hi @RRoundTable, thanks for making these changes. Can you fill out and email the CLA as described here?:

https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

Sorry. I checked late. I will fill out and email the CLA and let you know after that.

@matthewkotila
Copy link
Contributor

@matthewkotila: Hi @RRoundTable, thanks for making these changes. Can you fill out and email the CLA as described here?:
https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

@RRoundTable: Sorry. I checked late. I will fill out and email the CLA and let you know after that.

No worries 🙏

@RRoundTable
Copy link
Author

Hi. I send an email with CCLA. Thanks.

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.

2 participants