search

triton-inference-server / dali_backend

The Triton backend that allows running GPU-accelerated data pre-processing pipelines implemented in DALI's python API.
https://docs.nvidia.com/deeplearning/dali/user-guide/docs/index.html
MIT License
123 stars 29 forks source link

Auto-formatting #202

Open dyastremsky opened 1 year ago

dyastremsky commented 1 year ago

With the https://github.com/triton-inference-server/common/pull/94 in our formatter, we are running the formatter on all Triton repos. If you would like to use the formatting tool in this repo, feel free to approve/merge this PR.

szalpal commented 1 year ago

@dyastremsky , thank you for the contribution.

In dali_backend repo there's already a lint enabled (https://github.com/triton-inference-server/dali_backend/blob/main/cmake/lint.cmake). I'll run the CI with this PR to see how those comply and we'll decide then.

dali-automaton commented 1 year ago

CI MESSAGE: [8728848]: BUILD STARTED

dali-automaton commented 1 year ago

CI MESSAGE: [8728848]: BUILD PASSED

dyastremsky commented 1 year ago

@dyastremsky , thank you for the contribution.

In dali_backend repo there's already a lint enabled (https://github.com/triton-inference-server/dali_backend/blob/main/cmake/lint.cmake). I'll run the CI with this PR to see how those comply and we'll decide then.

In that case, feel free to close this PR at your discretion if preferred. For context, the formatting script used is here.

dyastremsky commented 1 year ago

@szalpal As a note, Triton is rolling out pre-commit hooks across our repos to automatically format instead . If you have interest in that, I'm happy to open a pull request for you or feel free to create one. You can see what it looks like here.