Repository navigation
refactor: use tensorrt-rs from kornia/tensorrt-rs instead of the in-tree trt-sys - #29
Merged
edgarriba merged 2 commits intoOct 4, 2026
Conversation
edgarriba
requested changes
Oct 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to kornia/tensorrt-rs#1. Removes the in-tree
crates/trt-sysand makesvrtusetensorrt-rsfrom kornia/tensorrt-rs instead.crates/trt-sys/deleted and dropped from the workspace.vrtdepends ontensorrt-rs = { git = "https://github.com/kornia/tensorrt-rs", branch = "main" }, imports aretensorrt_rs::now.trt-syspath patch removed from the fiveexamples/rtsp_*manifests, all six lockfiles refreshed (they resolve to688a1e8).-p trt-sysdropped from CI, docs and comments updated to the new name.No breaking changes for users of
vrtor the model crates.Testing
Ran the same steps on
main(d9e46b4) and on this branch against a real TensorRT, each from an empty engine cache.maincargo test -p vrt-xfeat --release --lib -- --ignoredcargo test -p vrt-raco-aliked --release --test gpu_fit -- --ignoredrfdetr_detect(ONNX -> engine -> inference)raco_detect(ONNX -> engine -> inference)All CI steps also pass locally in stub mode (fmt, check, both clippy runs, the three test steps).
System
Linux.x86_64-gnu.cuda-13.0)rust-toolchain.toml), gcc 15.2.0Env used for the real build:
Proof of run (this branch)
build: tensorrt-rs pulled from git, linked against real TensorRT
GPU tests
RF-DETR and RaCo-ALIKED on COCO val2017 000000039769 (two cats, two remotes)
mainprints the same detections, scores and boxes. ONNX files are the ones pinned invrt-hub(sha256 checked).Not covered
Invalid Node - node_Range_427). Same onmainand withtrtexec, so unrelated to this change. The XFeat GPU kernel tests above do pass.trt10.3.0.30because the build script can't read the version from the 10.13 headers and falls back. Same onmain.Use case: I did the trt-sys -> tensorrt-rs move and this is the second half of it.
🤖 Generated with Claude Code