search

cyrusbehr / tensorrt-cpp-api

TensorRT C++ API Tutorial
MIT License
577 stars 72 forks source link

Add cross-platform support #18

Closed xunzixunzi closed 10 months ago

xunzixunzi commented 1 year ago

Hey @cyrusbehr, I have successfully used your project on Windows 10, and I am eager to contribute to this project.

The project is useful, but it is not convenient when I integrate it into my project. To make it more user-friendly, I plan to modify the CMakeLists.txt and some functions. If you are okay with these changes, I will clean up my code and submit a pull request.

The compilation environment is as follows:

I conducted separate tests for semantic segmentation and object detection. Only the semantic segmentation using INT8 presision failed. You can find more details in the YOLOv8-TensorRT-CPP project's issue.

By the way, I am not a native English speaker. This issue may contain grammar errors. If there are any parts that are difficult to understand or if there are any mistakes, please let me know. This can also help me improve my English.

cyrusbehr commented 1 year ago

Hi there, yes you are more than welcome to contribute, I will give you credit in the readme as a collaborator. Please open up a PR

xunzixunzi commented 1 year ago

I was in a rush when I used it previously, so now I need to take some time to complete the code cleanup.

cyrusbehr commented 1 year ago

For sure, take your time.

cyrusbehr commented 11 months ago

Hey, any updates on this?

xunzixunzi commented 11 months ago

Yes, but I have been really busy with work these days, so I have not had the time to do thorough testing and write documentation. Perhaps I can submit a PR first and work on these tasks later？

cyrusbehr commented 10 months ago

Going to close this issue this the PR is up: https://github.com/cyrusbehr/tensorrt-cpp-api/pull/19

I still need to review the PR and integrate the changes.