nnstreamer / nnstreamer

:twisted_rightwards_arrows: Neural Network (NN) Streamer, Stream Processing Paradigm for Neural Network Apps/Devices.
https://nnstreamer.ai
GNU Lesser General Public License v2.1
683 stars 171 forks source link

Feature/upgrade tensorrt to 10 #4482

Closed bveldhoen closed 1 week ago

bveldhoen commented 2 weeks ago

PR Description

Updated tensorrt to version 10.0.1, including the required changes to the tensor_filter subplugin for tensorrt.

Changes proposed in this PR:

See also: https://github.com/nnstreamer/nnstreamer-example/pull/338

Self evaluation:

  1. Build test: [ ]Passed [ ]Failed [*]Skipped
  2. Run test: [ ]Passed [ ]Failed [*]Skipped

How to evaluate:

  1. Run the example in the nnstreamer-example.
bveldhoen commented 2 weeks ago

I've made the following changes:

myungjoo commented 2 weeks ago

After allowing the tensorrt 10+ subplugin be deployed along with older tensorrt subplugin and allowing its binaries be distinguishable against older tensorrt subplugin (by naming it differently), please do

  1. satisfy the github-action checkers (coding style and commit signing)
  2. remove unnecessary or unrelated changes
  3. squashing commits to remove unnecessary commits.
    • e.g., commit-1: new tensorrt subplugin, commit-2: docker updates, commit-3: test updates

Thank you so much for your contribution!

bveldhoen commented 1 week ago

Closed in favor of: https://github.com/nnstreamer/nnstreamer/pull/4489