Skip to content

Add guideline for building nuget package and example usage#2

Open
chilo-ms wants to merge 3 commits into
mainfrom
chi/add_csharp
Open

Add guideline for building nuget package and example usage#2
chilo-ms wants to merge 3 commits into
mainfrom
chi/add_csharp

Conversation

@chilo-ms
Copy link
Copy Markdown
Collaborator

Add guideline for building nuget package and providing C# example usage for TensorRT plugin EP.

Comment thread csharp/setup.bat
if defined PROCESSOR_ARCHITEW6432 set "ARCH=%PROCESSOR_ARCHITEW6432%"

if /i "%ARCH%"=="AMD64" (
set "DEST_EP_DLL_FOLDER=.\Microsoft.ML.OnnxRuntime.EP.TensorRT\runtimes\win-x64\native\"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

are we trying to create a nuget package that will be published, or just showing an example of how to create a nuget package?

if it's the former, would we want to support having both x64 and arm64 builds in one package?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

We are providing an example of how to create a nuget package at this moment.

We or others can update this setup.bat once there is a request for publishing a TensorRT plugin ep nuget package.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

hm, if all we want to do is provide an example, why not refer to this one instead?
https://github.com/microsoft/onnxruntime-inference-examples/tree/main/plugin_execution_providers/basic/csharp

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Eventually, the TensorRT plugin ep nuget package will be published either by us or Nvidia.
Let me add the support for x64 and arm64 builds.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants